-
Notifications
You must be signed in to change notification settings - Fork 12
resource(switch_port_settings): register the resource #475
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
base: main
Are you sure you want to change the base?
Conversation
8362811 to
0aa3e92
Compare
|
For context we merged #426 without registering the
We will merge this pull request once this resource becomes ready for customer usage or if we get the networking APIs to a point where we consider them stable for the near to mid future. |
0aa3e92 to
9b9a6de
Compare
be1fe18 to
8c17845
Compare
|
Ran into an issue with adding an address to an existing settings object |
|
Can you show me your Terraform configuration? I'll do a local test meanwhile. |
Registered the existing `oxide_switch_port_settings` resource and added public facing documentation and a changelog entry.
8c17845 to
f129c8c
Compare
|
Ah. The current code on the // Communities is include the provided communities in updates sent to the peer.
Communities []int `json:"communities,omitempty" yaml:"communities,omitempty"`My local testing shows the error as well. The default branch uses a Go SDK that serializes // Communities is include the provided communities in updates sent to the peer.
Communities []int `json:"communities" yaml:"communities"`I rebased onto You can pull the latest branch as well and try again. You should be good to go. |
Registered the existing
oxide_switch_port_settingsresource and added public facing documentation and a changelog entry.