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

Make queue first step of workflow #317

Closed
wants to merge 1 commit into from
Closed

Conversation

clrcrl
Copy link
Contributor

@clrcrl clrcrl commented Jan 4, 2021

Building on #316

This might help us out when we have the issue of two branches being updated at once, causing overlapping runs on a particular database.

The current failure message indicates that some work needs to be done on the CircleCI side. We might also need to add an API key to get their working, as per the docs

@clrcrl clrcrl mentioned this pull request Jan 4, 2021
3 tasks
@clrcrl clrcrl requested a review from jtcohen6 January 4, 2021 17:06
@jtcohen6
Copy link
Contributor

jtcohen6 commented Jan 6, 2021

Orb eddiewebb/[email protected] not loaded. To use this orb, an organization admin must opt-in to using third party orbs in Organization Security settings.

I think I have the permissions to turn this on, but I'd want to double check first.

Alternative thought:

  • Make the schema dbt_utils_integration_tests_redshift_{{ env_var('CIRCLE_PR_NUMBER') }}
  • Drop the schema as the final step of run_test.sh. (Or: Every few months I log into each database and drop a ton of schemata.)

@clrcrl
Copy link
Contributor Author

clrcrl commented Jan 6, 2021

I had that thought about that too, but I think if the orb works nicely, we should just use that. It feels a little less error-prone for some reason

Base automatically changed from update/circleci-workflows to master January 6, 2021 19:45
@clrcrl
Copy link
Contributor Author

clrcrl commented Apr 5, 2021

Closing due to lack of interest :)

@clrcrl clrcrl closed this Apr 5, 2021
@joellabes joellabes deleted the queue-circle-workflows branch October 21, 2021 22:33
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.

2 participants