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

[skip ci] Cirrus: Container for tracking image use #2213

Merged

Conversation

cevich
Copy link
Member

@cevich cevich commented Jan 23, 2019

Once built, this container can be utilized by automation to help keep
track of VM images. All parameters are passed in via env. vars.

Signed-off-by: Chris Evich [email protected]

@openshift-ci-robot openshift-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L labels Jan 23, 2019
@cevich cevich force-pushed the cirrus_imgts_container branch from c36b478 to 465ef4a Compare January 23, 2019 23:02
@cevich cevich changed the title WIP: Cirrus: Container for tracking image use [skip ci] WIP: Cirrus: Container for tracking image use Jan 23, 2019
@openshift-ci-robot openshift-ci-robot added size/M and removed do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L labels Jan 23, 2019
@cevich cevich force-pushed the cirrus_imgts_container branch 3 times, most recently from 9b7eff9 to e8c4e82 Compare January 23, 2019 23:32
Once built, this container can be utilized by automation to help keep
track of VM images.  All parameters are passed in via env. vars.

Signed-off-by: Chris Evich <[email protected]>
@cevich cevich changed the title [skip ci] WIP: Cirrus: Container for tracking image use [skip ci] Cirrus: Container for tracking image use Jan 24, 2019
@cevich cevich force-pushed the cirrus_imgts_container branch from e8c4e82 to 01e609d Compare January 24, 2019 13:47
@cevich
Copy link
Member Author

cevich commented Jan 24, 2019

@baude @mheon this PR only adds the container definition, PR #2214 actually runs it. I built the container in my registry for testing with that PR (it works): quay.io/cevich/imgts. PTAL, and when appropriate, could this be setup to build as quay.io/libpod/imgts:latest following the same pattern as for the gate image?

@baude
Copy link
Member

baude commented Feb 4, 2019

/approve

@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: baude, cevich

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 4, 2019
@baude
Copy link
Member

baude commented Feb 4, 2019

LGTM

@mheon
Copy link
Member

mheon commented Feb 4, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 4, 2019
@openshift-merge-robot openshift-merge-robot merged commit 465e19b into containers:master Feb 4, 2019
openshift-merge-robot added a commit that referenced this pull request Mar 7, 2019
Cirrus: Use imgts container to record metadata (needs #2213)
@cevich cevich deleted the cirrus_imgts_container branch June 30, 2021 18:01
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 22, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants