Skip to content

Conversation

@cardoe
Copy link

@cardoe cardoe commented Jan 27, 2026

Added an implementation to manage Network Segment Ranges which is a Neutron extension.

This depends on gophercloud/gophercloud#3614. For now it pulls that in with a replacement in go.mod.

Added an implementation to manage Network Segment Ranges which is a
Neutron extension.

This depends on gophercloud/gophercloud#3614. For now it pulls that in
with a replacement in go.mod.
@github-actions github-actions bot added the semver:major Breaking change label Jan 27, 2026
Copy link
Collaborator

@mandre mandre left a comment

Choose a reason for hiding this comment

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

Hey, thanks for the PR. I notice you've chosen the hard way decided to write the controller entirely by hand?

I strongly recommend you follow the docs at https://k-orc.cloud/development/overview/ and use the scaffolding tool. This ensures the code and the testing is consistent with the rest of the project. It will also make your life soooo much easier.

We'll also need e2e tests. We can't merge code unless it's properly tested in CI. You'll need to configure the devstack environment to have the necessary extension (similar to what we do in gophercloud).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:major Breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants