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

Support Kubernetes 1.11 #724

Merged
merged 5 commits into from
Aug 1, 2018
Merged

Support Kubernetes 1.11 #724

merged 5 commits into from
Aug 1, 2018

Conversation

ivan4th
Copy link
Contributor

@ivan4th ivan4th commented Jul 19, 2018

  • fix log directory handling
  • bump CRI Proxy to a version that supports k8s 1.8 .. 1.11
  • update tests to use k8s 1.11
  • fix missing CRD fields and reenable validation in demo.sh and cmd.sh
  • add CircleCI jobs for 1.9 and 1.10

This change is Reviewable

@ivan4th ivan4th force-pushed the ivan4th/k8s-1.11 branch 2 times, most recently from a24e6c2 to e2183bf Compare July 31, 2018 18:29
Ivan Shvedunov added 2 commits July 31, 2018 21:50
This fixes CRD validation problem during `kubectl apply`
in k8s 1.11
There seem to be flakes, concerning the multi-node config in particular.
@ivan4th ivan4th changed the title [WiP] Support Kubernetes 1.11 Support Kubernetes 1.11 Aug 1, 2018
Copy link
Contributor

@jellonek jellonek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 15 of 15 files at r1.
Reviewable status: 0 of 2 LGTMs obtained


.circleci/config.yml, line 367 at r1 (raw file):

  # e2e_1_9:
  #   <<: *e2e

Commented remains. Should be removed?


.circleci/config.yml, line 478 at r1 (raw file):

    #       tags:
    #         only:
    #           - /^v[0-9].*/

As above.

Copy link
Contributor

@jellonek jellonek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewable status: 0 of 2 LGTMs obtained, and 1 stale


.circleci/config.yml, line 367 at r1 (raw file):

Previously, jellonek (Piotr Skamruk) wrote…

Commented remains. Should be removed?

So it's because of flakes - ok.

Copy link
Contributor

@pigmej pigmej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 15 of 15 files at r1.
Reviewable status: 1 of 2 LGTMs obtained

Copy link
Contributor

@pigmej pigmej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewable status: 1 of 2 LGTMs obtained, and 1 stale

@pigmej pigmej merged commit cfd6816 into master Aug 1, 2018
@pigmej pigmej deleted the ivan4th/k8s-1.11 branch August 1, 2018 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants