Skip to content

Reverted json field name changes made in PR 79#82

Merged
mjsdell merged 1 commit into
mainfrom
revert-json-field-changes
Jan 10, 2025
Merged

Reverted json field name changes made in PR 79#82
mjsdell merged 1 commit into
mainfrom
revert-json-field-changes

Conversation

@santhoshatdell

Copy link
Copy Markdown
Contributor

PR Submission checklist

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
https://github.com/dell/csm/issues/1559

Common PR Checklist:

  • Have you made sure that the code compiles?
  • Have you commented your code, particularly in hard-to-understand areas
  • Did you run tests in a real Kubernetes cluster?
  • Have you maintained backward compatibility

Description of your changes:

Reverted json field name changes made in PR 79

@AronAtDell AronAtDell left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approving for automated dependabot changes only for CSM 1.13 release. Will not include any Engineering code additions.

@alikdell alikdell left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approving for automated dependabot changes only for CSM 1.13 release. Will not include any Engineering code additions.

@anandrajak1 anandrajak1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approving for automated dependabot changes only for CSM 1.13 release. Will not include any Engineering code additions.

@mjsdell mjsdell merged commit 0093b95 into main Jan 10, 2025
@mjsdell mjsdell deleted the revert-json-field-changes branch January 10, 2025 22:50
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.

5 participants