forked from uc-cdis/cloud-automation
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Pcdc dev #69
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* chore(awshelper-tf): Added terraform to awshelper image * chore(awshelper-tf): Added terraform to awshelper image * chore(awshelper-tf): Added terraform to awshelper image * chore(awshelper-tf): Added terraform to awshelper image * chore(awshelper-tf): Added terraform to awshelper image * chore(awshelper-tf): Added terraform to awshelper image * chore(awshelper-tf): Added terraform to awshelper image Co-authored-by: Edward Malinowski <edwardmalinowski@Eds-MacBook-Pro.attlocal.net>
Co-authored-by: John McCann <johnmccann@uchicago.edu>
uc-cdis#2041) * fix(jupyter-tests): Added scrape configs and modified prometheus command to check correct place * fix(jupyter-tests): Added scrape configs and modified prometheus command to check correct place Co-authored-by: Edward Malinowski <edwardmalinowski@Eds-MacBook-Pro.attlocal.net>
* Remove or hide deprecated logging features
* Updated Dockerfile Updated the Dockerfile for awshelper to use Ubuntu 22.04 as the base image. We're going to let the automated tests run as a sanity check, then go from there. * Made a change so that the we don't try to add a Google Cloud SDK repo for this specific version, which no longer seems to be supported. * Update cogwheel-register-client-job.yaml * Update variables.tf * Updated poetry install method, as the old one was deprecated * Added a fix for the older poetry install method, to see if everything will work. Co-authored-by: Hara Prasad <haraprasadj@uchicago.edu> Co-authored-by: cmlsn <100160785+cmlsn@users.noreply.github.com>
Co-authored-by: jawadqur <55899496+jawadqur@users.noreply.github.com>
* adding my key to ansible ssh keys for testing * adding a simplified playbook to add rsa keys to adminvms * updated hosts file to include new commons/vms * removed comment * updated hosts.yaml * making PE specific playbooks and made seperate updated hosts file * removed another entry in hosts that does not exist * removing a test public key from vms * correcting the path to keys * skipping gathering facts to speed up these playbooks * need gather facts to popular ansible_user_id var * adding Ajo's key Co-authored-by: Aidan Hilt <11202897+AidanHilt@users.noreply.github.com>
merge from CTDS
sync with master
Add BRH to ecr allowed accounts
…#2046) Co-authored-by: Edward Malinowski <edwardmalinowski@Eds-MacBook-Pro.attlocal.net>
* Updated most Terraform modules' variables files to the 1.0 specification, and improved inline documentation and comments. * Added more documentation around Terraform variables. * Added sample.tfvars for AWS modules, and provided documentation on most variables. The biggest gap currently is ami_account_id and csoc_account_id, which don't have good WHY documentation. Co-authored-by: jawadqur <55899496+jawadqur@users.noreply.github.com>
…cdis#1896) Co-authored-by: Edward Malinowski <edwardmalinowski@Eds-MacBook-Pro.attlocal.net>
…into uc-cdis-master
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.