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

locking cfa_azure version, adding state suffix to task ids #286

Merged
merged 1 commit into from
Oct 23, 2024

Conversation

arik-shurygin
Copy link
Collaborator

upgrading and locking cfa_azure version to commit hash: 5443bde or version 1.0.8. This upgrade adds the ability for us to specify suffix onto each task launched. I went ahead and added the state name as the default suffix to aid in interpretability when reading task lists in batch explorer.

This upgrade also changes some keys in the configuration toml, please reach out to me if you are getting weird errors when trying to launch your jobs and I can pass you the file in a safe way (it is not tracked on git for obvious reasons)

@@ -22,7 +22,7 @@ pandas = ">=2.0.3"
seaborn = ">=0.13.2"
shiny = ">=0.6.0"
pytest = ">=7.4.0"
cfa-azure = {git = "https://github.com/CDCgov/cfa_azure.git"}
cfa-azure = {git = "https://github.com/CDCgov/cfa_azure.git", rev="5443bde"}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would have prefer using tag="v1.0.6" or something but not fuss about it lol

Copy link
Collaborator

@kokbent kokbent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lg

@arik-shurygin arik-shurygin merged commit b3db61f into main Oct 23, 2024
3 checks passed
@arik-shurygin arik-shurygin deleted the cfa-azure-update branch October 23, 2024 15:49
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