Skip to content

When no argument passed to oc adm release new, dry run the output#22693

Merged
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
smarterclayton:messages
Apr 29, 2019
Merged

When no argument passed to oc adm release new, dry run the output#22693
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
smarterclayton:messages

Conversation

@smarterclayton
Copy link
Copy Markdown
Contributor

Print an output consistent with oc adm release info --verify if no
arguments are specified or if an image is pushed. Other modes bypass the
image creation and so do not have this output.

Make --dry-run on oc image append calculate the SHA instead of just doing
nothing.

Clean up some incremental output from append and in general make the entire
experience more pleasant.

Finally, perform a final step verification that a no-argument recreation
of the payload works correctly.

The error message when an image-reference points to a non-existent
image can be shortened and made more succinct since it is very
common.

Clean up the output of new to be easier to scan.
@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Apr 28, 2019
@openshift-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: smarterclayton

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

The pull request process is described here

Details 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 Apr 28, 2019
Print an output consistent with oc adm release info --verify if
no arguments are specified or if an image is pushed. Other modes
bypass the image creation and so do not have this output.

Make --dry-run on oc image append calculate the SHA instead of
just doing nothing.

Clean up some incremental output from append and in general make
the entire experience more pleasant.
@smarterclayton
Copy link
Copy Markdown
Contributor Author

/retest

@smarterclayton smarterclayton added the lgtm Indicates that a PR is ready to be merged. label Apr 28, 2019
@smarterclayton
Copy link
Copy Markdown
Contributor Author

Last piece required to audit releases.

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit 82881ec into openshift:master Apr 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants