Skip to content

Conversation

lwr20
Copy link
Member

@lwr20 lwr20 commented Oct 4, 2022

Use kubectl replace instead of apply when upgrading. If you don't, you hit:

The CustomResourceDefinition "installations.operator.tigera.io" is invalid: metadata.annotations: Too long: must have at most 262144 bytes

Description

Related issues/PRs

Todos

  • Tests
  • Documentation
  • Release note

Release Note

TBD

Reminder for the reviewer

Make sure that this PR has the correct labels and milestone set.

Every PR needs one docs-* label.

  • docs-pr-required: This change requires a change to the documentation that has not been completed yet.
  • docs-completed: This change has all necessary documentation completed.
  • docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

  • release-note-required: This PR has user-facing changes. Most PRs should have this label.
  • release-note-not-required: This PR has no user-facing changes.

Other optional labels:

  • cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
  • needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

Use kubectl replace instead of apply when upgrading.  If you don't, you hit:
```
The CustomResourceDefinition "installations.operator.tigera.io" is invalid: metadata.annotations: Too long: must have at most 262144 bytes
```
@lwr20 lwr20 requested a review from a team as a code owner October 4, 2022 15:51
@marvin-tigera marvin-tigera added this to the Calico v3.25.0 milestone Oct 4, 2022
@marvin-tigera marvin-tigera added docs-pr-required Change is not yet documented release-note-required Change has user-facing impact (no matter how small) labels Oct 4, 2022
@taxilian
Copy link

taxilian commented Oct 4, 2022

I hit this and had to ask about it on slack =] thanks @lwr20 for the quick explanation, also for telling me about kubectl replace which I somehow didn't know about

@caseydavenport caseydavenport added docs-not-required Docs not required for this change release-note-not-required Change has no user-facing impact labels Oct 10, 2022
@marvin-tigera marvin-tigera removed release-note-required Change has user-facing impact (no matter how small) docs-pr-required Change is not yet documented labels Oct 10, 2022
@caseydavenport caseydavenport merged commit 8f1ae21 into master Oct 10, 2022
@caseydavenport caseydavenport deleted the lwr-upg branch October 10, 2022 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-not-required Docs not required for this change release-note-not-required Change has no user-facing impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants