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

r/instance: unable to add hosts to management domain as day-n #209

Open
4 tasks done
pradeep17j opened this issue Jul 30, 2024 · 3 comments
Open
4 tasks done

r/instance: unable to add hosts to management domain as day-n #209

pradeep17j opened this issue Jul 30, 2024 · 3 comments
Labels
bug Bug enhancement Enhancement
Milestone

Comments

@pradeep17j
Copy link

pradeep17j commented Jul 30, 2024

Code of Conduct

  • I have read and agree to the Code of Conduct.
  • Vote on this issue by adding a 👍 reaction to the original issue initial description to help the maintainers prioritize.
  • Do not leave "+1" or other comments that do not add relevant information or questions.
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Terraform

1.9.2

Terraform Provider

0.10.0

VMware Cloud Foundation

5.2

Description

After successful bringup operation, adding hosts to a management cluster returns back success for both plan and apply. But when inspected on SDDC Manager it has not added any hosts. The terraform apply for this change returns in 0 seconds with success which is an indicator that it actually did not do any thing.

Affected Resources or Data Sources

r/vcf_instance

Terraform Configuration

main.tf and variable.tf:

https://gist.github.com/pradeep17j/7a6eab29a8320c103167e773f48455ae

Debug Output

https://gist.github.com/pradeep17j/e313961c8be7ce525e24886e7acdf8dd

Panic Output

No response

Expected Behavior

Successful add host to management cluster.

Actual Behavior

False success about adding hosts to mgmt cluster

Steps to Reproduce

Edit main.tf for resource vcf_instance to add an extra host to the list of existing hosts.

Environment Details

No response

Screenshots

No response

References

No response

@pradeep17j pradeep17j added bug Bug needs-triage Needs Triage labels Jul 30, 2024
@github-actions github-actions bot added the pending-review Pending Review label Jul 30, 2024
@tenthirtyam tenthirtyam changed the title Add host to mgmt domain does nothing Unable to add hosts to management domain Jul 30, 2024
@tenthirtyam
Copy link
Collaborator

tenthirtyam commented Jul 30, 2024

Hi @pradeep17j 👋 -

I have formatted the issue content and version updated to reflect supported and shipping versions.

We kindly ask that you do the same for future issues.

Ryan Johnson
Distinguished Engineer, VMware by Broadcom

@tenthirtyam tenthirtyam removed the pending-review Pending Review label Jul 30, 2024
@tenthirtyam tenthirtyam modified the milestones: 0.10.0, Backlog, v0.11.0 Jul 30, 2024
@tenthirtyam tenthirtyam added the enhancement Enhancement label Jul 30, 2024
@tenthirtyam tenthirtyam modified the milestones: v0.11.0, Backlog Jul 30, 2024
@tenthirtyam
Copy link
Collaborator

tenthirtyam commented Jul 31, 2024

This would only work for the initial run - r/instance is really more an imperative resource, rather than declarative.

cc @spacegospod @dimitarproynov

@tenthirtyam tenthirtyam removed the needs-triage Needs Triage label Jul 31, 2024
@tenthirtyam tenthirtyam changed the title Unable to add hosts to management domain r/instance: unable to add hosts to management domain as day-n Jul 31, 2024
Copy link

github-actions bot commented Oct 8, 2024

'Marking this issue as stale due to inactivity. This helps us focus on the active issues. If this issue receives no comments in the next 30 days it will automatically be closed.

If this issue was automatically closed and you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context.
Thank you!'

@github-actions github-actions bot added the stale Stale label Oct 8, 2024
@tenthirtyam tenthirtyam removed the stale Stale label Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug enhancement Enhancement
Projects
None yet
Development

No branches or pull requests

2 participants