This repository has been archived by the owner on Jun 28, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 2
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: patch bgpsession spec to remove status enum (#178)
While testing the new cluster API 1.5 changes in kubernetes-sigs/cluster-api-provider-packet#674 We discovered the API was returning down,down for the bgpsessionstatus after enabling BGP on a device. This didn't match any of the values in the enum and would return an error. @ctreatma recommends we remove this enum and go back to just a string. --------- Signed-off-by: Chris Privitere <23177737+cprivitere@users.noreply.github.com> Co-authored-by: Charles Treatman <ctreatman@equinix.com>
- Loading branch information
1 parent
df3a206
commit cd21e3a
Showing
9 changed files
with
45 additions
and
170 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
15 changes: 15 additions & 0 deletions
15
patches/spec.fetched.json/20231206-remove-bgp-session.patch
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
diff --git a/spec/oas3.patched/components/schemas/BgpSession.yaml b/spec/oas3.patched/components/schemas/BgpSession.yaml | ||
index db8baf8..50bcc22 100644 | ||
--- a/spec/oas3.patched/components/schemas/BgpSession.yaml | ||
+++ b/spec/oas3.patched/components/schemas/BgpSession.yaml | ||
@@ -28,10 +28,6 @@ properties: | ||
status will start with "unknown" and progress to "up" or "down" depending on | ||
the connected device. Subsequent "unknown" values indicate a problem acquiring | ||
status from the switch. ' | ||
- enum: | ||
- - unknown | ||
- - up | ||
- - down | ||
type: string | ||
updated_at: | ||
format: date-time |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters