Skip to content

Fix: Handle EN provisioning status before running AMT commands#307

Closed
damiankopyto wants to merge 1 commit intomainfrom
damian/amtfix
Closed

Fix: Handle EN provisioning status before running AMT commands#307
damiankopyto wants to merge 1 commit intomainfrom
damian/amtfix

Conversation

@damiankopyto
Copy link
Contributor

Description

Describe the purpose of this pull request.

Changes

List the changes you have made.

Additional Information

Include any additional information, such as how to test your changes.

Checklist

  • Tests passed
  • Documentation updated

host := *iresp.JSON200

if (powerFlag != "" || policyFlag != "") && host.Instance != nil {
if powerFlag != "" || policyFlag != "" {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

note to self: instead of deleting instance run a check for (host.Instance != || host.status***) this should give flexibility for both EMF provisioned and manually provsioned nodes

@damiankopyto damiankopyto changed the title Fix: Dont check for instance if editing amt related host fields Fix: Handle EN provisioning status before running AMT commands Feb 4, 2026
@damiankopyto
Copy link
Contributor Author

Closing in favor of #310

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments