Skip to content

update kube to 1.17#24335

Merged
openshift-merge-robot merged 29 commits intoopenshift:masterfrom
deads2k:rebase
Dec 20, 2019
Merged

update kube to 1.17#24335
openshift-merge-robot merged 29 commits intoopenshift:masterfrom
deads2k:rebase

Conversation

@deads2k
Copy link
Copy Markdown
Contributor

@deads2k deads2k commented Dec 19, 2019

#24286 plus two commits for skipping tests

@mfojtik @p0lyn0mial the skips I did here need to be reverted - bugzillas required.

Got ok from @smarterclayton for stability. Skew worked well, three more flakes driven out. We plan to merge over the build test ooms.

p0lyn0mial and others added 29 commits December 18, 2019 12:35
@p0lyn0mial
pin to origin-4.4-kubernetes-1.17.0-rc.2
5077039
@p0lyn0mial
pin other dependencies
055e17c
@p0lyn0mial
remove unused packages
37abb07
@p0lyn0mial
UPSTREAM: <drop>: bump(*)
9b828d0
@p0lyn0mial
UPSTREAM: <drop>: updates k/k bin data
003abcf
@p0lyn0mial
UPSTREAM: <drop>: update openapi generation
6571f71
@p0lyn0mial
UPSTREAM(node:review): <drop>: rename LegacyManager instead of bumpin…
af4011c 
…g opencontainers/runc
@p0lyn0mial
update references in origin/e2e files
ffbab88
@p0lyn0mial
Replace github.com/coreos/etcd by go.etcd.io/etcd
292c327
@p0lyn0mial
fix version in Docker image
7dcc8ec
@soltysh @p0lyn0mial
UPSTREAM: <carry>: skip openshift-apiserver served CRDs from crd-disc…
5c487f8 
…overy-available posthook
@p0lyn0mial
fixing etcd storage path test as EndpointSlice feature gate was disabled
a2b0565
@p0lyn0mial
fixing cmd test case as extensions/v1beta1 was disabled
cfa017d
@p0lyn0mial
disable sig-cloud-provider-gcp aka sig-cluster-lifecycle tests - unsu…
390e4e0 
…pported configuration
@p0lyn0mial
UPSTREAM: 86354: improves watch and report e2e test
417d3b6
@p0lyn0mial
etcd storage path tests add removed APIs
bfc9fad
@p0lyn0mial
disable EndpointSlices (Alpha) test
c6c8274
@p0lyn0mial
UPSTREAM: <carry>: ignore StorageState and StorageVersionMigration wh…
242d7d7 
…en checking custom columns (kubectl)
@p0lyn0mial
UPSTREAM: <carry>: GC doesn't ignore apps/deployments
9fcfb2e
@smarterclayton @p0lyn0mial
UPSTREAM: <carry>: Remove a redundant output in the tests
1970c57 
This line is not necessary for our test usage and should not be an
issue in OpenShift (openshift-tests already verifies this correctly).
@soltysh @p0lyn0mial
UPSTREAM: <carry>: Expose a simple journald shim on the kubelet logs …
a6de55a 
…endpoint
@rphillips @p0lyn0mial
UPSTREAM: 86009: kubelet: guarantee at most only one cinfo per contai…
aceffea 
…nerID
@p0lyn0mial
make master publish to origin-4.4-kubernetes-1.17.0-rc.2
7591ee5
@p0lyn0mial
bump test/conformance-k8s.sh to point to 1.17
be2b7df
@p0lyn0mial
UPSTREAM: <carry>: conditionally fill the UserAgent from the currentl…
c635a34 
…y running test

OpenShift uses these function before any test is run and they cause NPE
@p0lyn0mial
openshift-test: don't set the UserAgent to include the current test n…
41dea56 
…ame when loading cloud provider config
@deads2k
UPSTREAM: <drop>: skip shufflesharding test that takes too long
a690376
@deads2k
UPSTREAM: <drop>: skip PSP test that is inconsistently being skipped
ee647a9
@deads2k
fix router ingress fixture to use real apigroup
976c892
@openshift-ci-robot openshift-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Dec 19, 2019
@deads2k
Copy link
Copy Markdown
Contributor Author

deads2k commented Dec 19, 2019

/override ci/prow/e2e-gcp-builds
for known oom problems

/override ci/prow/e2e-aws-disruptive
for known bastion issues

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Dec 19, 2019

@deads2k: /override requires a failed status context to operate on.
The following unknown contexts were given:

  • ci/prow/e2e-aws-disruptive

Only the following contexts were expected:

  • ci/prow/artifacts
  • ci/prow/e2e-aws-fips
  • ci/prow/e2e-aws-serial
  • ci/prow/e2e-cmd
  • ci/prow/e2e-gcp
  • ci/prow/e2e-gcp-builds
  • ci/prow/e2e-gcp-upgrade
  • ci/prow/images
  • ci/prow/unit
  • ci/prow/verify
  • tide
Details

In response to this:

/override ci/prow/e2e-gcp-builds
for known oom problems

/override ci/prow/e2e-aws-disruptive
for known bastion issues

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
Copy Markdown

openshift-ci-robot commented Dec 19, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deads2k

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 approved Indicates a PR has been approved by an approver from all required OWNERS files. vendor-update Touching vendor dir or related files labels Dec 19, 2019
@deads2k deads2k added the lgtm Indicates that a PR is ready to be merged. label Dec 19, 2019
@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 19, 2019

/retest

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

@openshift openshift deleted a comment from openshift-ci-robot Dec 19, 2019
@deads2k
Copy link
Copy Markdown
Contributor Author

deads2k commented Dec 19, 2019 

level=info msg="Consuming Install Config from target directory"
level=info msg="Creating infrastructure resources..."
level=error
level=error msg="Error: Error applying IAM policy to project \"openshift-gce-devel-ci\": Too many conflicts.  Latest error: Error setting IAM policy for project \"openshift-gce-devel-ci\": googleapi: Error 409: There were concurrent policy changes. Please retry the whole read-modify-write with exponential backoff., aborted"
level=error
level=error msg="  on ../tmp/openshift-install-432914421/iam/main.tf line 11, in resource \"google_project_iam_member\" \"worker-storage-admin\":"
level=error msg="  11: resource \"google_project_iam_member\" \"worker-storage-admin\" {"

/retest

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 19, 2019

/retest

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

@deads2k
Copy link
Copy Markdown
Contributor Author

deads2k commented Dec 19, 2019

/retest

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 19, 2019

/retest

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

1 similar comment
@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 19, 2019

/retest

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

@smarterclayton
Copy link
Copy Markdown
Contributor

smarterclayton commented Dec 20, 2019

Wtf https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/24335/pull-ci-openshift-origin-master-e2e-gcp/4842

/retest

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 20, 2019

/retest

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

2 similar comments
@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 20, 2019

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 20, 2019

/retest

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

@openshift-merge-robot openshift-merge-robot merged commit 131dbb4 into openshift:master Dec 20, 2019
@p0lyn0mial p0lyn0mial mentioned this pull request Dec 20, 2019
Merged
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adambkaplan adambkaplan Awaiting requested review from adambkaplan

@ironcladlou ironcladlou Awaiting requested review from ironcladlou

Assignees

No one assigned

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/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. vendor-update Touching vendor dir or related files

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@deads2k @openshift-ci-robot @openshift-bot @smarterclayton @openshift-merge-robot @p0lyn0mial @soltysh @rphillips