Skip to content

test/aws: copy criotested cert if it was mounted by CI#11167

Closed
vrutkovs wants to merge 5 commits intoopenshift:devel-40from
vrutkovs:devel-40-opsmirror
Closed

test/aws: copy criotested cert if it was mounted by CI#11167
vrutkovs wants to merge 5 commits intoopenshift:devel-40from
vrutkovs:devel-40-opsmirror

Conversation

@vrutkovs
Copy link
Contributor

@vrutkovs vrutkovs commented Feb 11, 2019 •
edited
Loading

Switch to CRI-O from criotested repo.

This does the following:

  • Update MCD port
  • Wait for scaledup machines to become running
  • Avoid templating crio config - change only pause_image and pause_command
  • test/aws: copy ops-mirror.pem certificate to target systems to access the repo and install new CRIO

@openshift-ci-robot openshift-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Feb 11, 2019
@sdodson
Copy link
Member

sdodson commented Feb 11, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 11, 2019
mtnbikenc
mtnbikenc approved these changes Feb 11, 2019
View reviewed changes
Copy link
Member

@mtnbikenc mtnbikenc left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-bot
Copy link

openshift-bot commented Feb 11, 2019

/retest

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

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 11, 2019
@mtnbikenc
Copy link
Member

mtnbikenc commented Feb 12, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 12, 2019
@vrutkovs
Copy link
Contributor Author

vrutkovs commented Feb 12, 2019

/refresh

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 12, 2019
@mtnbikenc
Copy link
Member

mtnbikenc commented Feb 12, 2019

Looks like no changes in the new push.
/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 12, 2019
@vrutkovs
Copy link
Contributor Author

vrutkovs commented Feb 12, 2019

Right, had to force-push so that the new AWS secret would be picked up - works now

@vrutkovs vrutkovs mentioned this pull request Feb 12, 2019
Merged
@vrutkovs
Copy link
Contributor Author

vrutkovs commented Feb 12, 2019

Fixed the path to injected/dynamic

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 12, 2019
@mtnbikenc
Copy link
Member

mtnbikenc commented Feb 12, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 12, 2019
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 12, 2019
@mtnbikenc
Copy link
Member

mtnbikenc commented Feb 12, 2019

Hash change.
/lgtm

@vrutkovs
Copy link
Contributor Author

vrutkovs commented Feb 14, 2019

/test e2e-aws-scaleup

@vrutkovs
Copy link
Contributor Author

vrutkovs commented Feb 14, 2019 

Failing tests: 
[Conformance][Area:Networking][Feature:Router] The HAProxy router should enable openshift-monitoring to pull metrics [Suite:openshift/conformance/parallel/minimal] [sig-api-machinery] Namespaces [Serial] should ensure that all pods are removed when a namespace is deleted [Conformance] [Suite:openshift/conformance/serial/minimal] [Suite:k8s]

Router issue has occurred again, lets have another
/retest
and see if that happens again

@vrutkovs
Copy link
Contributor Author

vrutkovs commented Feb 14, 2019

/test e2e-aws-scaleup

1 similar comment
@vrutkovs
Copy link
Contributor Author

vrutkovs commented Feb 14, 2019

/test e2e-aws-scaleup

@vrutkovs
Copy link
Contributor Author

vrutkovs commented Feb 15, 2019 

Failing tests:

[Conformance][Area:Networking][Feature:Router] The HAProxy router should enable openshift-monitoring to pull metrics [Suite:openshift/conformance/parallel/minimal]
[Feature:DeploymentConfig] deploymentconfigs when run iteratively [Conformance] should only deploy the last deployment [Suite:openshift/conformance/parallel/minimal]

@vrutkovs
Copy link
Contributor Author

vrutkovs commented Feb 15, 2019

/retest

@vrutkovs vrutkovs mentioned this pull request Feb 15, 2019
Merged
@openshift-bot
Copy link

openshift-bot commented Feb 16, 2019

/retest

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

19 similar comments
@openshift-bot
Copy link

openshift-bot commented Feb 16, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 16, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 17, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 17, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 17, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 17, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 17, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 17, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 17, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 17, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 17, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 17, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 17, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 17, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 18, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 18, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 18, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 18, 2019

/retest

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

@openshift-bot
Copy link

openshift-bot commented Feb 18, 2019

/retest

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

@vrutkovs
Copy link
Contributor Author

vrutkovs commented Feb 18, 2019

Replaced by #11203

@vrutkovs vrutkovs closed this Feb 18, 2019
@openshift-ci-robot
Copy link

openshift-ci-robot commented Feb 18, 2019

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

Test name Commit Details Rerun command
ci/prow/e2e-aws-scaleup 47b2466 link /test e2e-aws-scaleup

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.

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

Reviewers

@mtnbikenc mtnbikenc mtnbikenc approved these changes

@michaelgugino michaelgugino Awaiting requested review from michaelgugino

Assignees

@sdodson sdodson

@mtnbikenc mtnbikenc

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@vrutkovs @sdodson @openshift-bot @mtnbikenc @openshift-ci-robot