[AUTO][OLM][OCP-21130] - [bug ALM-736]Fetching non-existent `PackageManifest` should return 404 by bandrade · Pull Request #24321 · openshift/origin

bandrade · 2019-12-18T14:00:41Z

PR Created to automate test case: @ecordell @njhale @kevinrizza. please have a review. Thanks! cc: @cuipinghuo @scolange @chengzhang1016 @emmajiafan @jianzhangbjz @tbuskey

Success execution

started: (0/1/1) "[Feature:Platform] OLM should Fetching non-existent `PackageManifest` does not return 404 [Suite:openshift/conformance/parallel]"

Dec 18 10:58:12.357: INFO: Waiting up to 30m0s for all (but 100) nodes to be schedulable
Dec 18 10:58:12.819: INFO: Waiting up to 10m0s for all pods (need at least 0) in namespace 'kube-system' to be running and ready
Dec 18 10:58:13.333: INFO: 0 / 0 pods in namespace 'kube-system' are running and ready (0 seconds elapsed)
Dec 18 10:58:13.333: INFO: expected 0 pod replicas in namespace 'kube-system', 0 are Running and Ready.
Dec 18 10:58:13.333: INFO: Waiting up to 5m0s for all daemonsets in namespace 'kube-system' to start
Dec 18 10:58:13.497: INFO: e2e test version: v0.0.0-master+$Format:%h$
Dec 18 10:58:13.648: INFO: kube-apiserver version: v1.16.2
Dec 18 10:58:13.806: INFO: Cluster IP family: ipv4
[BeforeEach] [Top Level]
  /home/bandrade/go/src/github.com/openshift/origin/test/extended/util/test.go:60
[BeforeEach] [Feature:Platform] OLM should
  /home/bandrade/go/src/github.com/openshift/origin/vendor/k8s.io/kubernetes/test/e2e/framework/framework.go:151
STEP: Creating a kubernetes client
[It] Fetching non-existent `PackageManifest` does not return 404 [Suite:openshift/conformance/parallel]
  /home/bandrade/go/src/github.com/openshift/origin/test/extended/operators/olm.go:110
Dec 18 10:58:13.853: INFO: Running 'oc --config=/home/bandrade/QE/ocp4-aws-ipi-vagrant-4.4/cluster/auth/kubeconfig get packagemanifest --all-namespaces --no-headers'
Dec 18 10:58:15.208: INFO: Running 'oc --config=/home/bandrade/QE/ocp4-aws-ipi-vagrant-4.4/cluster/auth/kubeconfig get packagemanifest a_package_that_not_exists -o yaml --loglevel=8'
Dec 18 10:58:16.062: INFO: Error running &{/usr/bin/oc [oc --config=/home/bandrade/QE/ocp4-aws-ipi-vagrant-4.4/cluster/auth/kubeconfig get packagemanifest a_package_that_not_exists -o yaml --loglevel=8] []   I1218 10:58:15.413644   24813 loader.go:359] Config loaded from file /home/bandrade/QE/ocp4-aws-ipi-vagrant-4.4/cluster/auth/kubeconfig
I1218 10:58:15.429269   24813 round_trippers.go:416] GET https://api.bandrade-888.qe.devcluster.openshift.com:6443/apis/packages.operators.coreos.com/v1/namespaces/default/packagemanifests/a_package_that_not_exists
I1218 10:58:15.429288   24813 round_trippers.go:423] Request Headers:
I1218 10:58:15.429294   24813 round_trippers.go:426]     Accept: application/json
I1218 10:58:15.429298   24813 round_trippers.go:426]     User-Agent: oc/v0.0.0 (linux/amd64) kubernetes/$Format
I1218 10:58:16.057924   24813 round_trippers.go:441] Response Status: 404 Not Found in 628 milliseconds
I1218 10:58:16.057949   24813 round_trippers.go:444] Response Headers:
I1218 10:58:16.057957   24813 round_trippers.go:447]     Audit-Id: a37296cf-6647-47aa-aa15-628a100fa206
I1218 10:58:16.057963   24813 round_trippers.go:447]     Cache-Control: no-cache, private
I1218 10:58:16.057969   24813 round_trippers.go:447]     Cache-Control: no-cache, private
I1218 10:58:16.057974   24813 round_trippers.go:447]     Content-Type: application/json
I1218 10:58:16.057979   24813 round_trippers.go:447]     Date: Wed, 18 Dec 2019 13:58:15 GMT
I1218 10:58:16.057984   24813 round_trippers.go:447]     Content-Length: 312
I1218 10:58:16.058015   24813 request.go:942] Response Body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"packagemanifests.packages.operators.coreos.com \"a_package_that_not_exists\" not found","reason":"NotFound","details":{"name":"a_package_that_not_exists","group":"packages.operators.coreos.com","kind":"packagemanifests"},"code":404}
I1218 10:58:16.058302   24813 helpers.go:196] server response object: [{
  "kind": "Status",
  "apiVersion": "v1",
  "metadata": {},
  "status": "Failure",
  "message": "packagemanifests.packages.operators.coreos.com \"a_package_that_not_exists\" not found",
  "reason": "NotFound",
  "details": {
    "name": "a_package_that_not_exists",
    "group": "packages.operators.coreos.com",
    "kind": "packagemanifests"
  },
  "code": 404
}]
F1218 10:58:16.058326   24813 helpers.go:114] Error from server (NotFound): packagemanifests.packages.operators.coreos.com "a_package_that_not_exists" not found
 I1218 10:58:15.413644   24813 loader.go:359] Config loaded from file /home/bandrade/QE/ocp4-aws-ipi-vagrant-4.4/cluster/auth/kubeconfig
I1218 10:58:15.429269   24813 round_trippers.go:416] GET https://api.bandrade-888.qe.devcluster.openshift.com:6443/apis/packages.operators.coreos.com/v1/namespaces/default/packagemanifests/a_package_that_not_exists
I1218 10:58:15.429288   24813 round_trippers.go:423] Request Headers:
I1218 10:58:15.429294   24813 round_trippers.go:426]     Accept: application/json
I1218 10:58:15.429298   24813 round_trippers.go:426]     User-Agent: oc/v0.0.0 (linux/amd64) kubernetes/$Format
I1218 10:58:16.057924   24813 round_trippers.go:441] Response Status: 404 Not Found in 628 milliseconds
I1218 10:58:16.057949   24813 round_trippers.go:444] Response Headers:
I1218 10:58:16.057957   24813 round_trippers.go:447]     Audit-Id: a37296cf-6647-47aa-aa15-628a100fa206
I1218 10:58:16.057963   24813 round_trippers.go:447]     Cache-Control: no-cache, private
I1218 10:58:16.057969   24813 round_trippers.go:447]     Cache-Control: no-cache, private
I1218 10:58:16.057974   24813 round_trippers.go:447]     Content-Type: application/json
I1218 10:58:16.057979   24813 round_trippers.go:447]     Date: Wed, 18 Dec 2019 13:58:15 GMT
I1218 10:58:16.057984   24813 round_trippers.go:447]     Content-Length: 312
I1218 10:58:16.058015   24813 request.go:942] Response Body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"packagemanifests.packages.operators.coreos.com \"a_package_that_not_exists\" not found","reason":"NotFound","details":{"name":"a_package_that_not_exists","group":"packages.operators.coreos.com","kind":"packagemanifests"},"code":404}
I1218 10:58:16.058302   24813 helpers.go:196] server response object: [{
  "kind": "Status",
  "apiVersion": "v1",
  "metadata": {},
  "status": "Failure",
  "message": "packagemanifests.packages.operators.coreos.com \"a_package_that_not_exists\" not found",
  "reason": "NotFound",
  "details": {
    "name": "a_package_that_not_exists",
    "group": "packages.operators.coreos.com",
    "kind": "packagemanifests"
  },
  "code": 404
}]
F1218 10:58:16.058326   24813 helpers.go:114] Error from server (NotFound): packagemanifests.packages.operators.coreos.com "a_package_that_not_exists" not found
 [] <nil> 0xc004920030 exit status 255 <nil> <nil> true [0xc003e96310 0xc003e96338 0xc003e96338] [0xc003e96310 0xc003e96338] [0xc003e96318 0xc003e96330] [0x9d9180 0x9d92b0] 0xc001b396e0 <nil>}:
I1218 10:58:15.413644   24813 loader.go:359] Config loaded from file /home/bandrade/QE/ocp4-aws-ipi-vagrant-4.4/cluster/auth/kubeconfig
I1218 10:58:15.429269   24813 round_trippers.go:416] GET https://api.bandrade-888.qe.devcluster.openshift.com:6443/apis/packages.operators.coreos.com/v1/namespaces/default/packagemanifests/a_package_that_not_exists
I1218 10:58:15.429288   24813 round_trippers.go:423] Request Headers:
I1218 10:58:15.429294   24813 round_trippers.go:426]     Accept: application/json
I1218 10:58:15.429298   24813 round_trippers.go:426]     User-Agent: oc/v0.0.0 (linux/amd64) kubernetes/$Format
I1218 10:58:16.057924   24813 round_trippers.go:441] Response Status: 404 Not Found in 628 milliseconds
I1218 10:58:16.057949   24813 round_trippers.go:444] Response Headers:
I1218 10:58:16.057957   24813 round_trippers.go:447]     Audit-Id: a37296cf-6647-47aa-aa15-628a100fa206
I1218 10:58:16.057963   24813 round_trippers.go:447]     Cache-Control: no-cache, private
I1218 10:58:16.057969   24813 round_trippers.go:447]     Cache-Control: no-cache, private
I1218 10:58:16.057974   24813 round_trippers.go:447]     Content-Type: application/json
I1218 10:58:16.057979   24813 round_trippers.go:447]     Date: Wed, 18 Dec 2019 13:58:15 GMT
I1218 10:58:16.057984   24813 round_trippers.go:447]     Content-Length: 312
I1218 10:58:16.058015   24813 request.go:942] Response Body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"packagemanifests.packages.operators.coreos.com \"a_package_that_not_exists\" not found","reason":"NotFound","details":{"name":"a_package_that_not_exists","group":"packages.operators.coreos.com","kind":"packagemanifests"},"code":404}
I1218 10:58:16.058302   24813 helpers.go:196] server response object: [{
  "kind": "Status",
  "apiVersion": "v1",
  "metadata": {},
  "status": "Failure",
  "message": "packagemanifests.packages.operators.coreos.com \"a_package_that_not_exists\" not found",
  "reason": "NotFound",
  "details": {
    "name": "a_package_that_not_exists",
    "group": "packages.operators.coreos.com",
    "kind": "packagemanifests"
  },
  "code": 404
}]
F1218 10:58:16.058326   24813 helpers.go:114] Error from server (NotFound): packagemanifests.packages.operators.coreos.com "a_package_that_not_exists" not found
Dec 18 10:58:16.062: INFO: Response: true
[AfterEach] [Feature:Platform] OLM should
  /home/bandrade/go/src/github.com/openshift/origin/test/extended/util/client.go:120
[AfterEach] [Feature:Platform] OLM should
  /home/bandrade/go/src/github.com/openshift/origin/vendor/k8s.io/kubernetes/test/e2e/framework/framework.go:152
Dec 18 10:58:16.064: INFO: Waiting up to 7m0s for all (but 100) nodes to be ready
Dec 18 10:58:16.530: INFO: Running AfterSuite actions on all nodes
Dec 18 10:58:16.530: INFO: Running AfterSuite actions on node 1

passed: (21.7s) 2019-12-18T13:58:16 "[Feature:Platform] OLM should Fetching non-existent `PackageManifest` does not return 404 [Suite:openshift/conformance/parallel]"

1 pass, 0 skip (21.7s)

bandrade · 2019-12-18T22:38:08Z

/retest

bandrade · 2019-12-19T02:26:56Z

/retest

kevinrizza

Had some comments and questions.

bandrade · 2019-12-23T16:53:46Z

/retest

bandrade · 2019-12-23T20:16:15Z

/retest

jianzhangbjz · 2019-12-24T07:55:47Z

/lgtm

bandrade · 2019-12-26T12:41:47Z

@shawn-hurley @bparees Could you help give approval? Thanks!

…anifest` should return 404

bandrade · 2020-01-15T23:50:55Z

/retest

bandrade · 2020-01-16T02:45:25Z

/retest

kevinrizza · 2020-01-16T14:15:31Z

/lgtm

shawn-hurley · 2020-01-16T14:28:37Z

/approve

openshift-ci-robot · 2020-01-16T14:29:23Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bandrade, jianzhangbjz, kevinrizza, shawn-hurley

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:

~~test/extended/OWNERS~~ [shawn-hurley]

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

openshift-bot · 2020-01-16T17:09:40Z

/retest

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

openshift-bot · 2020-01-16T17:22:38Z

/retest

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

openshift-bot · 2020-01-16T18:45:07Z

/retest

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

openshift-bot · 2020-01-16T19:19:32Z

/retest

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

openshift-bot · 2020-01-16T19:32:40Z

/retest

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

openshift-bot · 2020-01-18T15:58:31Z

/retest

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

openshift-bot · 2020-01-18T16:11:31Z

/retest

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

openshift-bot · 2020-01-18T16:24:31Z

/retest

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

openshift-bot · 2020-01-18T16:37:32Z

/retest

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

openshift-bot · 2020-01-18T16:50:33Z

/retest

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

openshift-bot · 2020-01-18T17:03:31Z

/retest

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

openshift-bot · 2020-01-18T17:16:32Z

/retest

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

shawn-hurley · 2020-01-18T17:17:56Z

/hold

This is just going to spam CI we can dive in on Monday

bandrade · 2020-01-20T13:49:29Z

/retest

bandrade · 2020-01-20T16:23:44Z

@shawn-hurley I think now we're good to proceed with the merge.

shawn-hurley · 2020-01-20T16:24:55Z

/retest
/hold cancel

openshift-bot · 2020-01-20T20:32:38Z

/retest

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

openshift-bot · 2020-01-20T21:37:48Z

/retest

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

openshift-bot · 2020-01-20T21:53:09Z

/retest

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

openshift-bot · 2020-01-20T22:28:44Z

/retest

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

openshift-bot · 2020-01-20T22:42:01Z

/retest

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

bandrade · 2020-01-20T23:41:53Z

/hold

bandrade · 2020-01-21T11:33:42Z

/retest

bandrade · 2020-01-22T12:09:54Z

/retest

bandrade · 2020-01-22T15:27:24Z

/retest
/hold cancel

shawn-hurley · 2020-01-22T15:31:05Z

/hold

I think that we want to follow the enhancement proposal here:

openshift/enhancements#183

and add this to the new repo:

https://github.com/openshift/extended-platform-tests

bandrade · 2020-01-22T17:07:24Z

created openshift/openshift-tests#6

openshift-ci-robot · 2020-01-22T19:02:01Z

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

Test name	Commit	Details	Rerun command
ci/prow/e2e-aws-serial	`73e9882`	link	`/test e2e-aws-serial`
ci/prow/e2e-cmd	`73e9882`	link	`/test e2e-cmd`
ci/prow/e2e-gcp	`73e9882`	link	`/test e2e-gcp`
ci/prow/e2e-gcp-upgrade	`73e9882`	link	`/test e2e-gcp-upgrade`

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.

bandrade · 2020-03-25T02:48:58Z

Closing since it was merged on openshift/openshift-tests#6 (comment)

openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 18, 2019

openshift-ci-robot requested review from deads2k and knobunc December 18, 2019 14:01

kevinrizza suggested changes Dec 20, 2019

View reviewed changes

Comment thread test/extended/operators/olm.go Outdated

Comment thread test/extended/operators/olm.go

bandrade changed the title ~~[AUTO][OLM][OCP-21130] - [bug ALM-736]Fetching non-existent PackageManifest does not return 404~~ [AUTO][OLM][OCP-21130] - [bug ALM-736]Fetching non-existent PackageManifest should return 404 Dec 23, 2019

bandrade force-pushed the ocp-21130 branch from ec9cf2c to 99c8eaa Compare December 23, 2019 14:21

openshift-ci-robot assigned jianzhangbjz Dec 24, 2019

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

[AUTO][OLM][OCP-21130] - [bug ALM-736]Fetching non-existent `PackageM…

73e9882

…anifest` should return 404

bandrade force-pushed the ocp-21130 branch from 99c8eaa to 73e9882 Compare January 15, 2020 18:04

openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jan 15, 2020

openshift-ci-robot assigned kevinrizza Jan 16, 2020

openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 16, 2020

openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 16, 2020

openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 18, 2020

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

openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 20, 2020

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

openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 22, 2020

bandrade closed this Mar 25, 2020

Conversation

bandrade commented Dec 18, 2019

Uh oh!

bandrade commented Dec 18, 2019

Uh oh!

bandrade commented Dec 19, 2019

Uh oh!

kevinrizza left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

bandrade commented Dec 23, 2019

Uh oh!

bandrade commented Dec 23, 2019

Uh oh!

jianzhangbjz commented Dec 24, 2019

Uh oh!

bandrade commented Dec 26, 2019

Uh oh!

bandrade commented Jan 15, 2020

Uh oh!

bandrade commented Jan 16, 2020

Uh oh!

kevinrizza commented Jan 16, 2020

Uh oh!

shawn-hurley commented Jan 16, 2020

Uh oh!

openshift-ci-robot commented Jan 16, 2020

Uh oh!

openshift-bot commented Jan 16, 2020

Uh oh!

openshift-bot commented Jan 16, 2020

Uh oh!

openshift-bot commented Jan 16, 2020

Uh oh!

openshift-bot commented Jan 16, 2020

Uh oh!

openshift-bot commented Jan 16, 2020

Uh oh!

openshift-bot commented Jan 18, 2020

Uh oh!

openshift-bot commented Jan 18, 2020

Uh oh!

openshift-bot commented Jan 18, 2020

Uh oh!

openshift-bot commented Jan 18, 2020

Uh oh!

openshift-bot commented Jan 18, 2020

Uh oh!

openshift-bot commented Jan 18, 2020

Uh oh!

openshift-bot commented Jan 18, 2020

Uh oh!

shawn-hurley commented Jan 18, 2020

Uh oh!

bandrade commented Jan 20, 2020

Uh oh!

bandrade commented Jan 20, 2020

Uh oh!

shawn-hurley commented Jan 20, 2020

Uh oh!

openshift-bot commented Jan 20, 2020

Uh oh!

openshift-bot commented Jan 20, 2020

Uh oh!

openshift-bot commented Jan 20, 2020

Uh oh!

openshift-bot commented Jan 20, 2020

Uh oh!

openshift-bot commented Jan 20, 2020

Uh oh!

bandrade commented Jan 20, 2020

Uh oh!

bandrade commented Jan 21, 2020

Uh oh!

bandrade commented Jan 22, 2020

Uh oh!

bandrade commented Jan 22, 2020

Uh oh!