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

SYSENG-1335: reserve available IP addresses #163

Merged
2 commits merged into from
Jun 10, 2024

Conversation

ghost
Copy link

@ghost ghost commented Jun 5, 2024

Description

Reserve an available address based on filters.

Checklist

  • added release notes to Unreleased section in CHANGELOG.md, if user facing change
  • release new version of go-anxcloud and update go.mod to use that version

References

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

@ghost ghost self-assigned this Jun 5, 2024
@ghost ghost force-pushed the SYSENG-1335/create-ip-address-in-prefix branch from 236394c to d8f725c Compare June 5, 2024 05:56
@ghost ghost changed the title SYSENG-1335: reserve available addresses SYSENG-1335: reserve available IP addresses Jun 5, 2024
@ghost ghost force-pushed the SYSENG-1335/create-ip-address-in-prefix branch from d8f725c to 9de898c Compare June 5, 2024 06:02
nachtjasmin
nachtjasmin previously approved these changes Jun 6, 2024
Copy link
Contributor

@nachtjasmin nachtjasmin left a comment

Choose a reason for hiding this comment

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

Pretty good work! Also, I'm really thankful that the docs are getting some love now! 👍

anxcloud/resource_ip_address.go Outdated Show resolved Hide resolved
go.mod Outdated Show resolved Hide resolved
@ghost ghost force-pushed the SYSENG-1335/create-ip-address-in-prefix branch 3 times, most recently from d3b4b61 to 8cb83f7 Compare June 6, 2024 10:25
@ghost ghost force-pushed the SYSENG-1335/create-ip-address-in-prefix branch from 8cb83f7 to f53c034 Compare June 6, 2024 11:56
@ghost ghost marked this pull request as ready for review June 6, 2024 11:58
@ghost ghost requested a review from nachtjasmin June 7, 2024 06:22
@@ -152,6 +152,8 @@ func TestAccAnxCloudVirtualServer(t *testing.T) {
}

func TestAccAnxCloudVirtualServerFromScratch(t *testing.T) {
t.Skip("specific from_scratch templates no longer available")
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure whether we should just remove the test in this case. I don't think it'll come back and even if, we can always revert the commit for that. ^^

Copy link
Author

Choose a reason for hiding this comment

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

There is a new more generic from_scratch template which is still somewhat WIP and could cause some issues that require manual intervention. I think if we remove it entirely, it will probably be overlooked in the future.

Copy link
Contributor

Choose a reason for hiding this comment

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

Hm, okay, let's keep it then! 👍

@ghost ghost merged commit b7aaba1 into main Jun 10, 2024
4 of 6 checks passed
@ghost ghost deleted the SYSENG-1335/create-ip-address-in-prefix branch June 10, 2024 05:48
This pull request was closed.
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.

1 participant