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

Allow setting target subnet and static IP on failover #5688

Merged
merged 20 commits into from Aug 7, 2020

Conversation

martenbohlin
Copy link
Contributor

This pull request adds some configurability of the network interface when a failover is done.

The previous implementation finished once the initial setup was done and was not waiting for the replication to finish. But to configure the NIC we need to wait for the replication to finish before we can do that, in this pull request it will always wait for the replication even if no changes has to be done and I am not sure if that is desirable or not. Thoughts?

@tombuildsstuff tombuildsstuff modified the milestones: v2.1.0, v2.2.0 Mar 11, 2020
@katbyte katbyte modified the milestones: v2.2.0, v2.3.0 Mar 18, 2020
@tombuildsstuff tombuildsstuff modified the milestones: v2.3.0, v2.4.0, v2.5.0 Mar 26, 2020
@tombuildsstuff tombuildsstuff modified the milestones: v2.5.0, v2.6.0 Apr 8, 2020
@tombuildsstuff tombuildsstuff modified the milestones: v2.6.0, v2.7.0 Apr 16, 2020
@martenbohlin
Copy link
Contributor Author

Hi, is there something I can do to increase the chance of this pull request to be accepted? If it is the dependency update that is the concern I can take a look and see if it is possible to implement this without the upgrade.

Thanks for maintaining a great product! /Mårten

@katbyte katbyte modified the milestones: v2.7.0, v2.8.0 Apr 23, 2020
@tombuildsstuff tombuildsstuff added this to the v2.17.0 milestone Jun 25, 2020
@katbyte katbyte modified the milestones: v2.17.0, v2.18.0, v2.19.0 Jul 3, 2020
@jackofallops jackofallops modified the milestones: v2.19.0, v2.20.0 Jul 16, 2020
Copy link
Member

@jackofallops jackofallops left a comment

Choose a reason for hiding this comment

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

Hi @martenbohlin
I tried to push the notes below to be able to merge this for today's release, not realising that the fork is in an Org not your user so I don't have rights. If you can address @katbyte 's last remaining comment and correct the typo I've called out, I think we should be good to merge after running the tests.

Thanks.

Co-authored-by: Steve <11830746+jackofallops@users.noreply.github.com>
@martenbohlin
Copy link
Contributor Author

@jackofallops I have fixed the typo you pointed out. I think I have fixed all of @katbyte 's comments, but let me know if you have spotted something I have missed. Thanks

@ghost ghost removed the waiting-response label Jul 23, 2020
@katbyte katbyte modified the milestones: v2.21.0, v2.22.0, v2.23.0 Jul 31, 2020
@jackofallops
Copy link
Member

Tests passing, failures are transient or intermittent unrelated items:
image

@jackofallops jackofallops merged commit dcc86e7 into hashicorp:master Aug 7, 2020
jackofallops added a commit that referenced this pull request Aug 7, 2020
@ghost
Copy link

ghost commented Aug 13, 2020

This has been released in version 2.23.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
    version = "~> 2.23.0"
}
# ... other configuration ...

@martenbohlin martenbohlin deleted the allow-setting-target-subnet branch August 13, 2020 17:20
@ghost
Copy link

ghost commented Sep 6, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!

@hashicorp hashicorp locked and limited conversation to collaborators Sep 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants