Skip to content

data/rhcos: Bump to rhcos-4.3/ 43.81.20191028.2#2547

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
sinnykumari:master
Oct 29, 2019
Merged

data/rhcos: Bump to rhcos-4.3/ 43.81.20191028.2#2547
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
sinnykumari:master

Conversation

@sinnykumari
Copy link
Copy Markdown
Contributor

@sinnykumari sinnykumari commented Oct 22, 2019

Generated with:

$ ./hack/update-rhcos-bootimage.py https://releases-art-rhcos.svc.ci.openshift.org/art/storage/releases/rhcos-4.3/43.80.20191022.1/x86_64/meta.json

This contains latest machine-config-daemon on host which includes
Karg Day1 support feature openshift/machine-config-operator#798

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

jcpowermac commented Oct 22, 2019

/test e2e-vsphere
not sure it will work 🤞

@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Oct 22, 2019

/approve
But I'd really like to bump to the RHEL 8.1 bootimages which should hopefully also be coming out today FWIW.

@abhinavdahiya
Copy link
Copy Markdown
Contributor

abhinavdahiya commented Oct 22, 2019

/test e2e-gcp

@abhinavdahiya
Copy link
Copy Markdown
Contributor

abhinavdahiya commented Oct 22, 2019

/test e2e-gcp-upi

@abhinavdahiya
Copy link
Copy Markdown
Contributor

abhinavdahiya commented Oct 22, 2019

/test e2e-metal

@abhinavdahiya
Copy link
Copy Markdown
Contributor

abhinavdahiya commented Oct 22, 2019

/test e2e-azure

@jcpowermac
Copy link
Copy Markdown
Contributor

jcpowermac commented Oct 22, 2019

/test e2e-vsphere
AWS timeout

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 22, 2019

/hold
Let's wait for next build in progress which includes RHEL 8.1 bootimages and we will update this PR

@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 Oct 22, 2019
@sinnykumari sinnykumari changed the title data/rhcos: Bump to rhcos-4.3/ 43.80.20191022.1 data/rhcos: Bump to rhcos-4.3/ 43.81.20191022.0 Oct 22, 2019
@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 22, 2019

/hold cancel
Updated with RHEL 8.1 bootimages

@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 Oct 22, 2019
@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Oct 22, 2019

/approve

@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Oct 22, 2019

FWIW this already passed the machine-os-content promotion:
https://openshift-gce-devel.appspot.com/build/origin-ci-test/logs/release-promote-openshift-machine-os-content-e2e-aws-4.3/2380/

Will promote sha256:474bbc1f291a88f32c0b0de8b74051c91981341acfa98d92a78433d88f15578d, current is sha256:e4981c8441614b292fc9983f0623362c96607dbb497e45fb866108e3024573ba
Name:       quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:474bbc1f291a88f32c0b0de8b74051c91981341acfa98d92a78433d88f15578d
Media Type: application/vnd.docker.distribution.manifest.v2+json
Created:    19m ago
Image Size: 651.9MB in 1 layers
Layers:     651.9MB sha256:0bbdb8b59bbc31896f0d60a320afea060f9fd4082b6d154ba3331ae9898a358e
OS:         linux
Arch:       amd64
Entrypoint: /noentry
Labels:     com.coreos.ostree-commit=70508e46194c2593a575b2660a6b5135d27fda05812cf505aeaad7e30b2b8b87
            version=43.81.20191022.0
...
2019/10/22 19:42:00 Ran for 1h35m47s
Tag ocp/4.3-art-latest:machine-os-content set to quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:474bbc1f291a88f32c0b0de8b74051c91981341acfa98d92a78433d88f15578d.

@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Oct 22, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 22, 2019
@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 23, 2019

/retest

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 23, 2019

/test e2e-vsphere

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 23, 2019

test e2e-gcp

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 23, 2019

/test e2e-gcp

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 23, 2019

/test e2e-azure

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 23, 2019

/test e2e-metal

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 23, 2019

/test e2e-aws-scaleup-rhel7

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 23, 2019

/test e2e-metal

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 23, 2019

/test e2e-azure

@yuqi-zhang
Copy link
Copy Markdown
Contributor

yuqi-zhang commented Oct 28, 2019

/retest

@iamemilio
Copy link
Copy Markdown

iamemilio commented Oct 28, 2019

/test e2e-openstack

@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Oct 28, 2019

@sinnykumari Please bump to 43.81.20191028.2 (still building) but this one already passed promotion here:
https://openshift-gce-devel.appspot.com/build/origin-ci-test/logs/release-promote-openshift-machine-os-content-e2e-aws-4.3/2563/

@yuqi-zhang
Copy link
Copy Markdown
Contributor

yuqi-zhang commented Oct 28, 2019

/retest
Giving the timing of openshift/cluster-kube-apiserver-operator#629 let's give this one last go

@sinnykumari
data/rhcos: Bump to rhcos-4.3/ 43.81.20191028.2
c18a2f6 
Generated with:

$ ./hack/update-rhcos-bootimage.py https://releases-art-rhcos.svc.ci.openshift.org/art/storage/releases/rhcos-4.3/43.81.20191028.2/x86_64/meta.json

This contains latest machine-config-daemon on host which includes
Karg Day1 support feature openshift/machine-config-operator#798
@sinnykumari sinnykumari changed the title data/rhcos: Bump to rhcos-4.3/ 43.81.20191025.3 data/rhcos: Bump to rhcos-4.3/ 43.81.20191028.2 Oct 29, 2019
@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 29, 2019

@sinnykumari Please bump to 43.81.20191028.2 (still building) but this one already passed promotion here:

Thanks, undated to 43.81.20191028.2

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 29, 2019

/test e2e-aws-scaleup-rhel7

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 29, 2019

/test e2e-metal

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 29, 2019

/test e2e-azure

@runcom
Copy link
Copy Markdown
Member

runcom commented Oct 29, 2019

/retest

@runcom
Copy link
Copy Markdown
Member

runcom commented Oct 29, 2019 •
edited
Loading

scaleup-rhel7 job is flacky as hell, and it's not related to the kargs change as the workers are rhel7 workers and we don't apply kargs there.

I'm not sure about metal and azure failures seems to be unrelated but aws works 👍

@runcom
Copy link
Copy Markdown
Member

runcom commented Oct 29, 2019

/approve

this is a prereq to land kargs day1 support in MCO also

@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Oct 29, 2019

Azure fell over with at least:

openshift-must-gather-xq4gv/must-gather-52vdv tolerates all taints

Which looks quite unrelated to this - something like a race condition where we're trying to must-gather at the same time as the test suite is executing?

And

StdErr: "buildconfig.build.openshift.io/ruby-sample-build-tc created\nbuildconfig.build.openshift.io/ruby-sample-build-ts created\nError from server (InternalError): Internal error occurred: admission plugin "build.openshift.io/BuildConfigSecretInjector" failed to complete mutation in 13s",

Which looks like a flake.

/test e2e-azure

@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Oct 29, 2019

Looking at https://openshift-release.svc.ci.openshift.org/ for 4.3.0-0.nightly which are "postsubmits/periodics" that are running on existing code, the metal suite has a pretty high flake rate.

Digging through those...it looks like all the failures here have been seen before.

/test e2e-metal

@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Oct 29, 2019

OK, we've got Azure passing. I think if this e2e-metal run fails, let's take a look and if it's still similar failures as without this PR, pull out the /override hammer?

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Oct 29, 2019

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

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

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.

@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Oct 29, 2019

/approve
/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 29, 2019
@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Oct 29, 2019

/assign @sdodson

@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Oct 29, 2019

Scott, can you or someone on the team approve?

@sdodson
Copy link
Copy Markdown
Member

sdodson commented Oct 29, 2019

/approve

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Oct 29, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, runcom, sdodson, sinnykumari

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 Oct 29, 2019
@openshift-merge-robot openshift-merge-robot merged commit 681c60a into openshift:master Oct 29, 2019
@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Oct 29, 2019

Note to us: e2e-metal was not actually marked as required yet, just an informing job. That said, I think it looks like if we fixed just one test its pass rate would go up a lot.

@sinnykumari
Copy link
Copy Markdown
Contributor Author

sinnykumari commented Oct 29, 2019

Thanks you all for getting this merged!

@mik-dass mik-dass mentioned this pull request May 6, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jcpowermac jcpowermac Awaiting requested review from jcpowermac

@patrickdillon patrickdillon Awaiting requested review from patrickdillon

Assignees

@sdodson sdodson

@cgwalters cgwalters

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.