Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rebase from openstack/kuryr-kubernetes #173

Merged
merged 2 commits into from Mar 5, 2020

Conversation

dulek
Copy link
Contributor

@dulek dulek commented Mar 5, 2020

No description provided.

There are several issues with the OpenShift job.

First of all it's running OpenShift 3.11, based on Kubernetes 1.11 which
was released almost 2 years ago. According to K8s deprecation policy [1]
the minimum support period for an API is just one year, so we might now
start to see some mismatches between supported features (as e.g. with
CRDs beta version, which just got deprecated, but stable version is not
supported by API 1.11).

Second of all - it's now constantly failing on every patch for an
unknown reason.

It's worth noting that upgrading the CI to use OpenShift 4.x is
extremely troublesome, as now OpenShift expects the installation to be
run with the full installer, which only supports nested deployments. And
nested deployments are really hard to do in the gate as we don't have
any guarantee of nested virtualization here.

This commit moves the OpenShift job to experimental queue.

[1] https://kubernetes.io/docs/reference/using-api/deprecation-policy/

Change-Id: Ibe683d008f26debd45bc0a0713d8141653860455
Change-Id: I4ac14c31c55bb4d1d77c822af10349df51ca70c5
@openshift-ci-robot openshift-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Mar 5, 2020
@gryf
Copy link
Member

gryf commented Mar 5, 2020

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Mar 5, 2020
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dulek, gryf

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:

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

@openshift-merge-robot openshift-merge-robot merged commit 768bfcc into openshift:master Mar 5, 2020
MaysaMacedo pushed a commit to MaysaMacedo/kuryr-kubernetes-1 that referenced this pull request Jul 1, 2021
Don't set a network status at all if we're not managing networking.
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. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants