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

fix: allow to use fixed mgmt IPs #63

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

titouan-joseph
Copy link
Contributor

@RavinderReddyF5 can you have a lok to this ?

We cannot assign fixed IPs on the management subnet

@RavinderReddyF5
Copy link
Collaborator

Hi @titouan-joseph , can we have scenario this PR going to address?

@titouan-joseph
Copy link
Contributor Author

Yes, we want to have a fixed IP for our management subnet but we cannot assign it.

Here how we have done the job

  mgmt_subnet_ids             = [{ "subnet_id" = aws_subnet.mgmt[count.index].id, "public_ip" = false, "private_ip_primary" = var.f5_mgmt_ip[count.index] }]

The var f5_mgmt_ip is ["10.10.10.4", "10.10.10.20"]

@titouan-joseph
Copy link
Contributor Author

The weird thing is that the local variable mgmt_private_ip_primary is defined but not used noware

@titouan-joseph
Copy link
Contributor Author

Hi @RavinderReddyF5,

I have finally found the last working version !

What we want to do : use a fixed IP for the management interfaces, we don't want public IP via EIP.
So we have this input for the mgmt subnet mgmt_subnet_ids = [{ "subnet_id" = [AWS subnet ID], "public_ip" = false, "private_ip_primary" = "10.10.10.10" }] and no private IPs were shown in the plan.

I think my PR should solve the problem..

Another point that I think it is weired, it is we switch the public_ip to true we cannot assign a static IP in the private network, I don't know if it is noramal or not.

@RavinderReddyF5
Copy link
Collaborator

Hi @titouan-joseph , i got into some issue while running changes. will merge PR once resoled those issue

@titouan-joseph
Copy link
Contributor Author

Got it !
Which issue did you had ?

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.

2 participants