Skip to content

Conversation

@sjenning
Copy link
Contributor

@sjenning sjenning commented Sep 14, 2022

4.12 images are not published yet. 4.11 is the latest.

Pulls fix for slow catalog start from operator-framework/operator-lifecycle-manager#2791

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 14, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sjenning

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 openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 14, 2022
@enxebre
Copy link
Member

enxebre commented Sep 14, 2022

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 14, 2022
@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 5dd049f and 2 for PR HEAD 817b361 in total

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 950ea39 and 1 for PR HEAD 817b361 in total

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD ce5bba7 and 0 for PR HEAD 817b361 in total

@openshift-ci-robot
Copy link

/hold

Revision 817b361 was retested 3 times: holding

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 16, 2022
@sjenning
Copy link
Contributor Author

/retest-required

@sjenning
Copy link
Contributor Author

/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 26, 2022
@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD b27a93f and 2 for PR HEAD 817b361 in total

@sjenning
Copy link
Contributor Author

/hold
catalog pods are terminating for some reason

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 26, 2022
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Sep 26, 2022
@sjenning
Copy link
Contributor Author

/hold cancel
/retest-required

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 26, 2022
@sjenning
Copy link
Contributor Author

/retest-required

@sjenning
Copy link
Contributor Author

triggering the no-op detector

{"level":"info","ts":"2022-09-27T00:51:53Z","msg":"WARNING: Object got updated more than one time without a no-op update, this indicates hypershift incorrectly reverting defaulted values","type":"*v1.Deployment","name":"e2e-clusters-4dckm-example-vvtwq/certified-operators-catalog","diff":"  &v1.Deployment{\n  \tTypeMeta:   {Kind: \"Deployment\", APIVersion: \"apps/v1\"},\n  \tObjectMeta: {Name: \"certified-operators-catalog\", Namespace: \"e2e-clusters-4dckm-example-vvtwq\", UID: \"df1a53e8-7b2c-4e3a-ab64-8d0c5f7170d8\", ResourceVersion: \"26949\", ...},\n  \tSpec: v1.DeploymentSpec{\n  \t\tReplicas: &1,\n  \t\tSelector: &{MatchLabels: {\"olm.catalogSource\": \"certified-operators\"}},\n  \t\tTemplate: v1.PodTemplateSpec{\n  \t\t\tObjectMeta: {Labels: {\"app\": \"certified-operators-catalog\", \"hypershift.openshift.io/control-plane-component\": \"certified-operators-catalog\", \"hypershift.openshift.io/hosted-control-plane\": \"e2e-clusters-4dckm-example-vvtwq\", \"olm.catalogSource\": \"certified-operators\"}, Annotations: {\"hypershift.openshift.io/release-image\": \"registry.build04.ci.openshift.org/ci-op-thhl0hmb/release@sha256:\"...}},\n  \t\t\tSpec: v1.PodSpec{\n  \t\t\t\tVolumes:        nil,\n  \t\t\t\tInitContainers: nil,\n  \t\t\t\tContainers: []v1.Container{\n  \t\t\t\t\t{\n  \t\t\t\t\t\t... // 11 identical fields\n  \t\t\t\t\t\tLivenessProbe:  &{ProbeHandler: {Exec: &{Command: {\"grpc_health_probe\", \"-addr=:50051\"}}}, InitialDelaySeconds: 10, TimeoutSeconds: 1, PeriodSeconds: 10, ...},\n  \t\t\t\t\t\tReadinessProbe: &{ProbeHandler: {Exec: &{Command: {\"grpc_health_probe\", \"-addr=:50051\"}}}, InitialDelaySeconds: 5, TimeoutSeconds: 5, PeriodSeconds: 10, ...},\n  \t\t\t\t\t\tStartupProbe: &v1.Probe{\n  \t\t\t\t\t\t\t... // 2 identical fields\n  \t\t\t\t\t\t\tTimeoutSeconds:                1,\n  \t\t\t\t\t\t\tPeriodSeconds:                 10,\n- \t\t\t\t\t\t\tSuccessThreshold:              1,\n+ \t\t\t\t\t\t\tSuccessThreshold:              0,\n  \t\t\t\t\t\t\tFailureThreshold:              15,\n  \t\t\t\t\t\t\tTerminationGracePeriodSeconds: nil,\n  \t\t\t\t\t\t},\n  \t\t\t\t\t\tLifecycle:              nil,\n  \t\t\t\t\t\tTerminationMessagePath: \"/dev/termination-log\",\n  \t\t\t\t\t\t... // 6 identical fields\n  \t\t\t\t\t},\n  \t\t\t\t},\n  \t\t\t\tEphemeralContainers: nil,\n  \t\t\t\tRestartPolicy:       \"Always\",\n  \t\t\t\t... // 31 identical fields\n  \t\t\t},\n  \t\t},\n  \t\tStrategy:        {Type: \"RollingUpdate\", RollingUpdate: &{MaxUnavailable: &{Type: 1, StrVal: \"25%\"}, MaxSurge: &{Type: 1, StrVal: \"25%\"}}},\n  \t\tMinReadySeconds: 0,\n  \t\t... // 3 identical fields\n  \t},\n  \tStatus: {ObservedGeneration: 1, Replicas: 1, UpdatedReplicas: 1, UnavailableReplicas: 1, ...},\n  }\n","semanticDeepEqual":false,"updateCount":5}

@sjenning
Copy link
Contributor Author

failed due to #1764

@csrwng
Copy link
Contributor

csrwng commented Oct 5, 2022

/retest-required

@csrwng
Copy link
Contributor

csrwng commented Oct 5, 2022

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 5, 2022
@openshift-merge-robot openshift-merge-robot merged commit 2d3622f into openshift:main Oct 5, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 5, 2022

@sjenning: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-kubevirt-gcp-ovn 817b361 link false /test e2e-kubevirt-gcp-ovn
ci/prow/capi-provider-agent-sanity 1361f23 link false /test capi-provider-agent-sanity
ci/prow/kubevirt-e2e-kubevirt-gcp-ovn 1361f23 link false /test kubevirt-e2e-kubevirt-gcp-ovn
ci/prow/kubevirt-e2e-kubevirt-azure-ovn 1361f23 link false /test kubevirt-e2e-kubevirt-azure-ovn
ci/prow/e2e-aws 1361f23 link unknown /test e2e-aws

Full PR test history. Your PR dashboard.

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.

@csrwng
Copy link
Contributor

csrwng commented Oct 5, 2022

/cherry-pick release-4.11

@openshift-cherrypick-robot

@csrwng: new pull request created: #1786

Details

In response to this:

/cherry-pick release-4.11

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.

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants