Skip to content

Conversation

@cgwalters
Copy link
Member

@cgwalters cgwalters commented Nov 22, 2019

This has fixes for encryption, see e.g.:

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 22, 2019
@sdodson
Copy link
Member

sdodson commented Nov 24, 2019

/test e2e-metal
It will likely fail due to other problems but I'm curious.

@sdodson
Copy link
Member

sdodson commented Nov 24, 2019

/test e2e-gcp
/test e2e-azure

@sdodson sdodson mentioned this pull request Nov 24, 2019
2 tasks
@cgwalters
Copy link
Member Author

Leasing
/test e2e-gcp

Some sort of networking issue, probably real but not likely related to this
/test e2e-azure

@jcpowermac
Copy link
Contributor

/test e2e-vsphere

@jcpowermac
Copy link
Contributor

vSphere is failing on the usual tests. The install completed bootstrap and cluster complete. lgtm

@cgwalters
Copy link
Member Author

/test e2e-azure

1 similar comment
@cgwalters
Copy link
Member Author

/test e2e-azure

@cgwalters
Copy link
Member Author

Looks like a transient Prow issue
/test e2e-azure

@stbenjam
Copy link
Member

stbenjam commented Nov 26, 2019

/label platform/baremetal

Is this working?

Edit: No, it's not. I kicked off baremetal manually

@openshift-ci-robot
Copy link
Contributor

@stbenjam: The label(s) /label platform/baremetal cannot be applied. These labels are supported: platform/aws, platform/azure, platform/baremetal, platform/google, platform/libvirt, platform/openstack, ga

Details

In response to this:

/label platform/baremetal

Is this working?

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.

@openshift-ci-robot
Copy link
Contributor

@stbenjam: The label(s) /label platform/baremetal cannot be applied. These labels are supported: platform/aws, platform/azure, platform/baremetal, platform/google, platform/libvirt, platform/openstack, ga

Details

In response to this:

/label platform/baremetal

Is this working?

Edit: No, it's not. I kicked off [baremetal manually](http://10.8.144.11:8080/job/dev-tools/1332.

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.

@openshift-ci-robot
Copy link
Contributor

@stbenjam: The label(s) /label platform/baremetal cannot be applied. These labels are supported: platform/aws, platform/azure, platform/baremetal, platform/google, platform/libvirt, platform/openstack, ga

Details

In response to this:

/label platform/baremetal

Is this working?

Edit: No, it's not. I kicked off baremetal manually

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.

@metal3ci
Copy link

Build FAILURE, see build http://10.8.144.11:8080/job/dev-tools/1332/

@cgwalters
Copy link
Member Author

/test e2e-azure

@patrickdillon
Copy link
Contributor

Seeing

[systemd]
Failed Units: 1
  multipathd.service

in GCP. Seems like this should help that as well...

@cgwalters
Copy link
Member Author

Horray, have a ✔️ e2e-azure. Let's get this in?

@abhinavdahiya
Copy link
Contributor

/retest

does it solve the metal failures?

@cgwalters
Copy link
Member Author

does it solve the metal failures?

No, because at least the machines we're getting from Packet.net only have a TPM1.2 - and we don't make it easy to disable the encryption requirement on the bootstrap. See openshift/enhancements#98
(There are other packet issues being worked through in #2695 )

@abhinavdahiya
Copy link
Contributor

No, because at least the machines we're getting from Packet.net only have a TPM1.2 -

but the PR description says that it's fixing that bug...? Can we remove that if that's not the case.

/cc @wking
we might have to bump the AMI for upi tests

@cgwalters
Copy link
Member Author

but the PR description says that it's fixing that bug...? Can we remove that if that's not the case.

Sure, done.

@abhinavdahiya abhinavdahiya changed the title rhcos: Bump to 43.81.201911221453.0 Bug 1775728: rhcos: Bump to 43.81.201911221453.0 Nov 26, 2019
@openshift-ci-robot
Copy link
Contributor

@cgwalters: This pull request references Bugzilla bug 1775728, which is invalid:

  • expected the bug to target the "4.4.0" release, but it targets "4.3.0" instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

Details

In response to this:

Bug 1775728: rhcos: Bump to 43.81.201911221453.0

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.

@openshift-ci-robot openshift-ci-robot added the bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. label Nov 26, 2019
@abhinavdahiya
Copy link
Contributor

/bugzilla refresh

@openshift-bot
Copy link
Contributor

/retest

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

1 similar comment
@openshift-bot
Copy link
Contributor

/retest

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

@cgwalters
Copy link
Member Author

Known busted
/override e2e-metal
/override e2e-aws-scaleup-rhel7
Known to have some failing tests
/override e2e-vsphere

@openshift-ci-robot
Copy link
Contributor

@cgwalters: cgwalters unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file.

Details

In response to this:

Known busted
/override e2e-metal
/override e2e-aws-scaleup-rhel7
Known to have some failing tests
/override e2e-vsphere

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.

@openshift-bot
Copy link
Contributor

/retest

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

14 similar comments
@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
Contributor

/retest

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

@openshift-bot
Copy link
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 254319c into openshift:master Nov 27, 2019
@openshift-ci-robot
Copy link
Contributor

@cgwalters: All pull requests linked via external trackers have merged. Bugzilla bug 1775728 has been moved to the MODIFIED state.

Details

In response to this:

Bug 1775728: rhcos: Bump to 43.81.201911221453.0

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.

@openshift-cherrypick-robot

@abhinavdahiya: new pull request created: #2724

Details

In response to this:

/cherrypick release-4.3

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.

@openshift-ci-robot
Copy link
Contributor

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

Test name Commit Details Rerun command
ci/prow/e2e-metal a8e9991 link /test e2e-metal
ci/prow/e2e-aws-scaleup-rhel7 a8e9991 link /test e2e-aws-scaleup-rhel7
ci/prow/e2e-vsphere a8e9991 link /test e2e-vsphere

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.

wking added a commit to wking/openshift-release that referenced this pull request Dec 3, 2019
…1.201911221453.0

Catching up with openshift/installer@a8e9991876 (rhcos: Bump to
43.81.201911221453.0, 2019-11-22, openshift/installer#2714) and its
encryption fixes.

Unfortunately we still don't have a way to expose the installer's
recommended default, and I'm still opposed to reaching into the
installer sources and looking in its internal rhcos.json like
c85599a (templates/installer/e2e-metal: use the rhcos.json from
upi-installer image to pick bootimage, 2019-04-26, openshift#3612) is doing for
metal, because that's not what our users will be doing.  Using the
installer's tip AMI defaults is also not what users will be doing, but
I'm not clear on how we can select the right branch in the templates
to pull from the openshift-docs defaults [1,2,3] (which is what users
will be doing).

[1]: https://docs.openshift.com/container-platform/4.1/installing/installing_aws_user_infra/installing-aws-user-infra.html#installation-aws-user-infra-rhcos-ami_installing-aws-user-infra
[2]: https://docs.openshift.com/container-platform/4.2/installing/installing_aws_user_infra/installing-aws-user-infra.html#installation-aws-user-infra-rhcos-ami_installing-aws-user-infra
[3]: https://github.com/openshift/openshift-docs/blob/b24fe4684f8f6ba11cae92c5a779abbe4af57717/modules/installation-aws-user-infra-rhcos-ami.adoc#op-system-amis-for-the-aws-infrastructure
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. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. 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.