[release-4.3] Bug 1779995: fix(packageserver): cache packagemanifests #1181

openshift-cherrypick-robot · 2019-12-09T17:21:18Z

This is an automated cherry-pick of #1150

/assign njhale

openshift-ci-robot · 2019-12-09T17:21:21Z

@openshift-cherrypick-robot: This pull request references Bugzilla bug 1779990, which is invalid:

expected the bug to target the "4.3.0" release, but it targets "4.4.0" instead
expected the bug to be in one of the following states: NEW, ASSIGNED, ON_DEV, POST, POST, but it is MODIFIED 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.

In response to this:

[release-4.3] Bug 1779990: fix(packageserver): cache packagemanifests

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.

njhale · 2019-12-09T17:48:23Z

/retest

njhale · 2019-12-09T17:50:30Z

/bugzilla refresh

njhale · 2019-12-09T17:52:10Z

BZ automation is having some trouble RN: https://coreos.slack.com/archives/CEKNRGF25/p1575911986238000

njhale · 2019-12-09T17:52:35Z

/retest

njhale · 2019-12-10T05:27:38Z

/retest

ecordell · 2019-12-10T13:13:04Z

/lgtm

waiting for verification in master

openshift-ci-robot · 2019-12-10T13:13:12Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ecordell, openshift-cherrypick-robot

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

~~OWNERS~~ [ecordell]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

shawn-hurley · 2019-12-10T19:47:00Z

/hold

shawn-hurley · 2019-12-10T20:02:07Z

/hold cancel

shawn-hurley · 2019-12-10T20:02:12Z

/retest

njhale · 2019-12-10T20:25:50Z

/retest

njhale · 2019-12-11T15:58:18Z

/retest

ecordell · 2019-12-11T16:40:28Z

/retest

njhale · 2019-12-11T19:39:30Z

Looks like there are some potentially real unit test flakes, and more concerning, real console e2e tests failing. Investigating now...

/hold
/retest

spadgett · 2019-12-11T21:40:36Z

/retest

njhale · 2019-12-11T23:30:55Z

@spadgett and I had a discussion on slack around this -- haven't found a root cause yet.

/retest

njhale · 2019-12-11T23:34:37Z

Re-ran the test of another 4.3 PR to see if the failure is disjoint from this patch.

njhale · 2019-12-12T04:48:04Z

/retest

njhale · 2019-12-12T12:55:51Z

Looks like the other backport failing on the same thing: #1179 (comment)

njhale · 2019-12-12T13:54:24Z

/retest

njhale · 2019-12-12T13:55:56Z

/bugzilla refresh

openshift-ci-robot · 2019-12-12T13:56:01Z

@njhale: This pull request references Bugzilla bug 1779995, which is valid. The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

/bugzilla refresh

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.

ecordell · 2019-12-12T16:44:52Z

/retest

ecordell · 2019-12-12T18:59:51Z

/hold cancel

njhale · 2019-12-12T22:34:47Z

/retest

njhale · 2019-12-13T15:26:20Z

/retest

njhale · 2019-12-13T17:01:33Z

/bugzilla refresh

openshift-ci-robot · 2019-12-13T17:05:52Z

@njhale: This pull request references Bugzilla bug 1779995, which is valid. The bug has been moved to the POST state.

In response to this:

/bugzilla refresh

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.

ecordell · 2019-12-13T17:28:54Z

/refresh

njhale · 2019-12-13T19:34:34Z

/retest

ecordell · 2019-12-13T19:51:48Z

/retest

openshift-ci-robot · 2019-12-13T20:56:23Z

@openshift-cherrypick-robot: All pull requests linked via external trackers have merged. Bugzilla bug 1779995 has been moved to the MODIFIED state.

In response to this:

[release-4.3] Bug 1779995: fix(packageserver): cache packagemanifests

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.

ecordell · 2020-01-09T15:01:42Z

/cherry-pick release-4.2

openshift-cherrypick-robot · 2020-01-09T15:01:49Z

@ecordell: #1181 failed to apply on top of branch "release-4.2":

error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	go.mod
M	go.sum
M	pkg/package-server/provider/registry_test.go
M	vendor/modules.txt
Falling back to patching base and 3-way merge...
Auto-merging vendor/modules.txt
Auto-merging pkg/package-server/provider/registry_test.go
Auto-merging go.sum
CONFLICT (content): Merge conflict in go.sum
Auto-merging go.mod
CONFLICT (content): Merge conflict in go.mod
Patch failed at 0001 fix(packageserver): cache packagemanifests

In response to this:

/cherry-pick release-4.2

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.

njhale added 3 commits December 9, 2019 17:21

fix(packageserver): cache packagemanifests

75ade1b

chore(modules): vendor deps

181445a

chore(git): remove and ignore packageserver test key pair

65edb4d

openshift-ci-robot assigned njhale Dec 9, 2019

openshift-cherrypick-robot mentioned this pull request Dec 9, 2019

Bug 1779990: fix(packageserver): cache packagemanifests #1150

Merged

5 tasks

openshift-ci-robot added the bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. label Dec 9, 2019

openshift-ci-robot requested review from ecordell and jpeeler December 9, 2019 17:21

openshift-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Dec 9, 2019

njhale changed the title ~~[release-4.3] Bug 1779990: fix(packageserver): cache packagemanifests~~ [release-4.3] Bug 1779995: fix(packageserver): cache packagemanifests Dec 9, 2019

This comment has been minimized.

Sign in to view

exdx mentioned this pull request Dec 9, 2019

[release-4.3] Bug 1780755: add resource limits to all OLM pods #1179

Merged

openshift-ci-robot assigned ecordell Dec 10, 2019

openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Dec 10, 2019

openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 10, 2019

shawn-hurley added bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. and removed bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. labels Dec 10, 2019

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 10, 2019

openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 10, 2019

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 removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 12, 2019

openshift-merge-robot merged commit bda7125 into operator-framework:release-4.3 Dec 13, 2019

crawford mentioned this pull request Jan 20, 2020

Bug 1787597: fix(packageserver): cache packagemanifests #1240

Merged

[release-4.3] Bug 1779995: fix(packageserver): cache packagemanifests #1181

[release-4.3] Bug 1779995: fix(packageserver): cache packagemanifests #1181

Conversation

openshift-cherrypick-robot commented Dec 9, 2019

openshift-ci-robot commented Dec 9, 2019

njhale commented Dec 9, 2019

This comment has been minimized.

njhale commented Dec 9, 2019

This comment has been minimized.

njhale commented Dec 9, 2019

njhale commented Dec 9, 2019

njhale commented Dec 10, 2019

ecordell commented Dec 10, 2019

openshift-ci-robot commented Dec 10, 2019

shawn-hurley commented Dec 10, 2019

shawn-hurley commented Dec 10, 2019

shawn-hurley commented Dec 10, 2019

njhale commented Dec 10, 2019

njhale commented Dec 11, 2019

ecordell commented Dec 11, 2019

njhale commented Dec 11, 2019

spadgett commented Dec 11, 2019

njhale commented Dec 11, 2019 • edited Loading

njhale commented Dec 11, 2019

njhale commented Dec 12, 2019

njhale commented Dec 12, 2019

njhale commented Dec 12, 2019

njhale commented Dec 12, 2019

openshift-ci-robot commented Dec 12, 2019

ecordell commented Dec 12, 2019

ecordell commented Dec 12, 2019

njhale commented Dec 12, 2019

njhale commented Dec 13, 2019

njhale commented Dec 13, 2019

openshift-ci-robot commented Dec 13, 2019

ecordell commented Dec 13, 2019

njhale commented Dec 13, 2019

ecordell commented Dec 13, 2019

openshift-ci-robot commented Dec 13, 2019

ecordell commented Jan 9, 2020

openshift-cherrypick-robot commented Jan 9, 2020

njhale commented Dec 11, 2019 •

edited

Loading