Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trying to update our CI and lock it down at the same time #3669

Draft
wants to merge 34 commits into
base: main
Choose a base branch
from

Conversation

vevetron
Copy link
Contributor

Description

Describe your changes and why you're making them. Please include the context, motivation, and relevant dependencies.

Resolves #[issue]

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

How has this been tested?

Include commands/logs/screenshots as relevant.

If making changes to dbt models, please run the command poetry run dbt run -s CHANGED_MODEL and poetry run dbt test -s CHANGED_MODEL, then include the output in this section of the PR.

Post-merge follow-ups

Document any actions that must be taken post-merge to deploy or otherwise implement the changes in this PR (for example, running a full refresh of some incremental model in dbt). If these actions will take more than a few hours after the merge or if they will be completed by someone other than the PR author, please create a dedicated follow-up issue and link it here to track resolution.

  • No action required
  • Actions required (specified below)

Copy link

github-actions bot commented Jan 30, 2025

The following changes will be applied to the production Kubernetes cluster upon merge.

BE AWARE this may not reveal changes that have been manually applied to the cluster getting undone—applying manual changes to the cluster should be avoided.

No manifest changes found for prod.

@vevetron vevetron force-pushed the git_action_testground branch from ff9ad62 to 6dfa645 Compare January 30, 2025 23:42
@vevetron
Copy link
Contributor Author

vevetron commented Feb 3, 2025

First part:
#3671

@vevetron vevetron changed the title Pollute the git space Trying to update our CI and lock it down at the same time Feb 3, 2025
@vevetron vevetron marked this pull request as ready for review February 4, 2025 18:24
@vevetron vevetron requested a review from evansiroky as a code owner February 4, 2025 18:24
@vevetron vevetron marked this pull request as draft February 4, 2025 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant