Skip to content
This repository was archived by the owner on Mar 6, 2026. It is now read-only.

Fix workflow vars#34

Draft
emmyoop wants to merge 3 commits into
get-select:mainfrom
emmyoop:er/testing-CI
Draft

Fix workflow vars#34
emmyoop wants to merge 3 commits into
get-select:mainfrom
emmyoop:er/testing-CI

Conversation

@emmyoop
Copy link
Copy Markdown
Contributor

@emmyoop emmyoop commented Dec 2, 2024

Github does not allow anything stored as a secret in their UI to be passed as a variable.

This change uses variables instead of secrets for user, role, database and warehouse.

Note: This requires a change in the repo settings for these values. They need to be set as repository variables instead of respository secrets.

@emmyoop emmyoop marked this pull request as draft December 2, 2024 17:32
@emmyoop emmyoop had a problem deploying to Approve Integration Tests December 2, 2024 17:32 — with GitHub Actions Failure
@emmyoop emmyoop had a problem deploying to Approve Integration Tests December 2, 2024 17:34 — with GitHub Actions Failure
@emmyoop emmyoop changed the title add comment Fix workflow vars Dec 2, 2024
@emmyoop emmyoop had a problem deploying to Approve Integration Tests December 2, 2024 17:39 — with GitHub Actions Failure
@emmyoop
Copy link
Copy Markdown
Contributor Author

emmyoop commented Dec 2, 2024

Do you want ci_dbt_text_package.yml to trigger off pull_request (current config) or pull_request_target (matches ci_test_package.yml)?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant