Skip to content

Comments

4.3 OKD: run e2e-aws test before promoting it to fcos namespace#5867

Closed
vrutkovs wants to merge 9 commits intoopenshift:masterfrom
vrutkovs:4.3-okd-promotion
Closed

4.3 OKD: run e2e-aws test before promoting it to fcos namespace#5867
vrutkovs wants to merge 9 commits intoopenshift:masterfrom
vrutkovs:4.3-okd-promotion

Conversation

@vrutkovs
Copy link
Contributor

Before new machine-os-content is tagged into fcos namespace it should be tested.

This PR creates a temporary release with pending machine-os-content and runs e2e-aws test.

Requires openshift/installer#2548

/cc @smarterclayton

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 12, 2019
@vrutkovs vrutkovs force-pushed the 4.3-okd-promotion branch 2 times, most recently from 6bf8bb0 to 28f2502 Compare November 12, 2019 14:18
@vrutkovs
Copy link
Contributor Author

/hold

until installer PR is merged

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 12, 2019
@vrutkovs
Copy link
Contributor Author

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 12, 2019
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: vrutkovs
To complete the pull request process, please assign smarterclayton
You can assign the PR to them by writing /assign @smarterclayton in a comment when ready.

The full list of commands accepted by this bot can be found 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 size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Dec 10, 2019
ci-operator \
--artifact-dir=$(ARTIFACTS) \
--secret-dir=/usr/local/pull-secret \
--secret-dir=/usr/local/e2e-$(CLUSTER_TYPE)-cluster-profile \
Copy link
Contributor

Choose a reason for hiding this comment

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

Set target

@vrutkovs
Copy link
Contributor Author

/hold

Needs openshift/machine-config-operator#1311 to be merged and commits squashed

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 11, 2019
@openshift-ci-robot
Copy link
Contributor

@vrutkovs: The following tests failed, say /retest to rerun them all:

Test name Commit Details Rerun command
ci/rehearse/promote-release-openshift-okd-machine-os-content-e2e-aws-4.3 86ade0e link /test pj-rehearse
ci/prow/pj-rehearse 86ade0e link /test pj-rehearse
ci/prow/step-registry-shellcheck 86ade0e link /test step-registry-shellcheck

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@vrutkovs vrutkovs closed this Jan 7, 2020
@vrutkovs
Copy link
Contributor Author

vrutkovs commented Jan 7, 2020

Closed in favor of #6559

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants