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 undeployment on destroy for resource mso_schema_site. (DCNE-55) #293

Merged
merged 3 commits into from
Oct 3, 2024

Conversation

gmicol
Copy link
Collaborator

@gmicol gmicol commented Sep 18, 2024

Fixes #287

The Delete function will first check if template is deployed on site before being undeployed. If not deployed, It will now properly skip the undeployment phase which previously led to the resource being removed from State but not from MSO.

@gmicol gmicol added the jira-sync Sync this issue to Jira label Sep 18, 2024
@gmicol gmicol self-assigned this Sep 18, 2024
@github-actions github-actions bot changed the title Fix undeployment on destroy for resource mso_shcema_site Fix undeployment on destroy for resource mso_shcema_site (DCNE-55) Sep 18, 2024
@gmicol gmicol changed the title Fix undeployment on destroy for resource mso_shcema_site (DCNE-55) Fix undeployment on destroy for resource mso_shcema_site. (DCNE-55) Sep 18, 2024
@gmicol gmicol changed the title Fix undeployment on destroy for resource mso_shcema_site. (DCNE-55) Fix undeployment on destroy for resource mso_schema_site. (DCNE-55) Sep 18, 2024
@gmicol gmicol force-pushed the mso_schema_site-deletion-bugfix branch from ffca548 to 9c8df05 Compare September 18, 2024 23:42
akinross
akinross previously approved these changes Sep 19, 2024
Copy link
Collaborator

@akinross akinross left a comment

Choose a reason for hiding this comment

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

LGTM

shrsr
shrsr previously approved these changes Sep 20, 2024
Copy link
Collaborator

@shrsr shrsr left a comment

Choose a reason for hiding this comment

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

LGTM

samiib
samiib previously approved these changes Sep 23, 2024
Copy link
Collaborator

@samiib samiib left a comment

Choose a reason for hiding this comment

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

LGTM

sajagana
sajagana previously approved these changes Oct 1, 2024
Copy link
Collaborator

@sajagana sajagana left a comment

Choose a reason for hiding this comment

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

LGTM!

@gmicol gmicol dismissed stale reviews from sajagana, samiib, shrsr, and akinross via 76a6527 October 1, 2024 16:29
Copy link
Collaborator

@sajagana sajagana left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Collaborator

@anvitha-jain anvitha-jain left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@akinross akinross left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@shrsr shrsr left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@lhercot lhercot left a comment

Choose a reason for hiding this comment

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

LGTM

@lhercot lhercot merged commit d47c982 into CiscoDevNet:master Oct 3, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jira-sync Sync this issue to Jira
Projects
None yet
Development

Successfully merging this pull request may close these issues.

mso_schema_site delete does not work when schema is not saved. (DCNE-55)
7 participants