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

Update Helm 3 installation instructions for pre-1.15 clusters #1263

Merged
merged 1 commit into from Feb 15, 2020

Conversation

muvaf
Copy link
Member

@muvaf muvaf commented Feb 13, 2020

Description of your changes

The details are in #1170 . Long story short, Crossplane could not be installed to pre-1.15 clusters if CRDs had to be validated, which was the case in Helm 3 until 3.1 release which has a flag --disable-openapi-validation.

Fixes #1170

How has this code been tested?

I've tested this in a Kubernetes 1.13.11 cluster which is default version in GKE.

Checklist

I have:

  • Run make reviewable to ensure this PR is ready for review.
  • Ensured this PR contains a neat, self documenting set of commits.
  • Updated any relevant documentation and examples.
  • Reported all new error conditions into the log or as an event, as
    appropriate.

For more about what we believe makes a pull request complete, see our
definition of done.

@muvaf muvaf requested a review from hasheddan February 13, 2020 20:47
@upbound-bot
Copy link
Collaborator

68% (0.0%) vs master 68%

Copy link
Member

@hasheddan hasheddan left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @muvaf

@muvaf muvaf requested a review from jbw976 February 14, 2020 09:34
```console
kubectl create namespace crossplane-system
helm repo add crossplane-alpha https://charts.crossplane.io/alpha

# post-1.15 cluster
Copy link
Member

Choose a reason for hiding this comment

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

should this be 1.15 and after, or 1.15+ cluster? the two headers of pre 1.15 and post 1.15 aren't clear on which bucket 1.15 falls into

Copy link
Member Author

Choose a reason for hiding this comment

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

Just updated to make it more clear

…clusters

Signed-off-by: Muvaffak Onus <onus.muvaffak@gmail.com>
@upbound-bot
Copy link
Collaborator

68% (+0.43%) vs master 67%

@sonarcloud
Copy link

sonarcloud bot commented Feb 15, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@jbw976 jbw976 merged commit 152303e into crossplane:master Feb 15, 2020
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.

Cannot install Crossplane to Kubernetes versions lower than 1.15 with Helm 3
4 participants