Skip to content

Conversation

@chatton
Copy link
Contributor

@chatton chatton commented Apr 21, 2021

This PR adds stages to inventory.yaml which will upload the generated Dockerfiles to the dockerfile s3 bucket.

@chatton chatton changed the base branch from master to CLOUDP-83092_add_context_image_for_agents April 21, 2021 15:54
inventory.yaml Outdated

labels:
quay.expires-after: 48h
quay.expires-after: $(inputs.params.expire_after)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

when releasing, we want to specify "Never", not "48h"

Base automatically changed from CLOUDP-83092_add_context_image_for_agents to master April 23, 2021 16:39

args = {

def _build_agent_args(config: DevConfig) -> Dict[str, str]:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

these values are shared in both agent images.

@chatton chatton marked this pull request as ready for review April 26, 2021 09:23
@chatton chatton requested a review from bznein April 26, 2021 09:23
Copy link
Contributor

@bznein bznein left a comment

Choose a reason for hiding this comment

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

LGTM!

@chatton chatton merged commit 1bf5f80 into master Apr 26, 2021
@chatton chatton deleted the CLOUDP-83092_upload_dockerfiles_to_s3 branch April 26, 2021 10:46
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.

3 participants