-
Notifications
You must be signed in to change notification settings - Fork 50
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
Migrate release AMI gc to ci-automation #1952
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We probably could just drop this whole directory, no? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We would have to migrate the |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,11 +1 @@ | ||
The scripts in this directory are run from [OS Jenkins jobs][jenkins-os]. By | ||
storing the Jenkins scripts in this repository, they are more tightly coupled | ||
to the release branch of the SDK scripts that they require. The Jenkins jobs | ||
are responsible for setting up the environment and securely initializing an SDK | ||
in the workspace before running these scripts. | ||
|
||
The special files named `formats-${BOARD}.txt` are space-separated lists of VM | ||
image formats that should be built for releases on this branch; i.e. the script | ||
`vm.sh` is run for each item in the list. | ||
|
||
[jenkins-os]: https://github.com/coreos/jenkins-os | ||
This folder is unused. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suppose that these are yet different stuff from
AWS_ACCESS_KEY_ID
andAWS_SECRET_ACCESS_KEY
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think yes, the test and release credentials are different