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

[artifact-manager] [archetypes/vra] (#210) Cloud Template folder name and the name from details.json mismatch issue fixed in vRA #300

Merged
merged 5 commits into from
Jul 10, 2024

Conversation

sugeeshC
Copy link
Contributor

@sugeeshC sugeeshC commented Jul 5, 2024

Description

Cloud Template folder name and details.json->name mismatch issue fixed.

Previous Behavior

  • When Cloud-Template folder name and details.json->name mismatch, folder name is used for validating if the record exists on the server and details.json->name is used for creating/updating the cloud template on server.

Current Behavior

  • When Cloud-Template folder name and details.json->name mismatch, now an error throwing explaining the mismatch.

  • details.json->name is now used for both validating and creating/updating cloud template on the server.

Checklist

  • I have added relevant error handling and logging messages to help troubleshooting
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation, relevant usage information (if applicable)
  • I have updated the PR title with affected component, related issue number and a short summary of the changes introduced
  • I have added labels for implementation type (enhancement, bug, dependencies) and version change (major, minor, patch)
  • I have tested against live environment, if applicable
  • I have synced any structure and/or content vRA-NG improvements with vra-ng and ts-vra-ng archetypes (if applicable)
  • I have my changes rebased and squashed to the minimal number of relevant commits. Notice: don't squash all commits
  • I have added a descriptive commit message with a short title, including a Fixed #XXX - or Closed #XXX - prefix to auto-close the issue

Testing

1.) Create vra 8 project using build-tools-for-vmware-aria.
2.) Create blueprint folder(Test-Blueprint-1) and content.yaml, details.json files accordingly.
3.) Add the blueprint folder name(Test-Blueprint-1) in project content.yaml.
4.) Change the details.json->name value into different from blueprint folder name. (Test-Blueprint-2)
5.) Push project to vra server.

Release Notes

[artifact-manager] (#210) Cloud Template folder name and details.json->name mismatch issue fixed in vra.

Related issues and PRs

Closed #210

Additional Notes

Please note that, this fix is applying only when, we have a 'blueprint folder' named as in the project 'content.yaml'.

Examples

  • Scenario 1 - Fix will be applied in this scenario.

content.yaml

blueprint:
  - Test-1

blueprints/Test-1/details.json

{
  "name": "Test-2",
  "description": "",
  "requestScopeOrg": false
}
  • Scenario 2 - Fix would not applied in following scenario. ( Since there is no 'Test-1' folder, blueprint will be skipped. )

content.yaml

blueprint:
  - Test-1

blueprints/Test-2/details.json

{
  "name": "Test-1",
  "description": "",
  "requestScopeOrg": false
}

For the second scenario, this is the current behavior as well.

Signed-off-by: sugeesh chandraweera <csugeesh@vmware.com>
Signed-off-by: sugeesh chandraweera <csugeesh@vmware.com>
@sugeeshC sugeeshC requested a review from a team as a code owner July 5, 2024 08:24
@github-actions github-actions bot added the kind/bug Something isn't working label Jul 5, 2024
@sugeeshC
Copy link
Contributor Author

sugeeshC commented Jul 5, 2024

Please note that, this fix is applying only when, we have a 'blueprint folder' named as in the project 'content.yaml'.

eg -

  • scenario 1 - Fix will be applied in this scenario.

content.yaml

blueprint:
  - Test-1

blueprints/Test-1/details.json

{
  "name": "Test-2",
  "description": "",
  "requestScopeOrg": false
}
  • scenario 2 - Fix would not applied in following scenario. ( Since there is no 'Test-1' folder, blueprint will be skipped. )

content.yaml

blueprint:
  - Test-1

blueprints/Test-2/details.json

{
  "name": "Test-1",
  "description": "",
  "requestScopeOrg": false
}

For the second scenario, this is the existing behavior as well.

@akantchev akantchev changed the title [artifact-manager] (#210) Cloud Template folder name and details.json->name mismatch issue fixed in vra. [artifact-manager] [archetypes/vra] (#280) Cloud Template folder name and details.json -> name mismatch issue fixed in vRA Jul 5, 2024
@sugeeshC sugeeshC changed the title [artifact-manager] [archetypes/vra] (#280) Cloud Template folder name and details.json -> name mismatch issue fixed in vRA [artifact-manager] [archetypes/vra] (#210) Cloud Template folder name and details.json -> name mismatch issue fixed in vRA Jul 5, 2024
@VenelinBakalov VenelinBakalov added the version/patch The change is a non-breaking bugfix label Jul 8, 2024
Signed-off-by: sugeesh chandraweera <csugeesh@vmware.com>
@sugeeshC sugeeshC changed the title [artifact-manager] [archetypes/vra] (#210) Cloud Template folder name and details.json -> name mismatch issue fixed in vRA [artifact-manager] [archetypes/vra] (#210) Cloud Template folder name and the name from details.json mismatch issue fixed in vRA Jul 9, 2024
Signed-off-by: sugeesh chandraweera <csugeesh@vmware.com>
@vmwclabot
Copy link
Member

@sugeeshC, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <john.doe@email.org> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

1 similar comment
@vmwclabot
Copy link
Member

@sugeeshC, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <john.doe@email.org> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

Signed-off-by: sugeesh chandraweera <csugeesh@vmware.com>
@sugeeshC sugeeshC force-pushed the fix/210-blueprint-folder-name-mismatch-issue branch from b615015 to 17ed27e Compare July 10, 2024 10:57
@vmwclabot
Copy link
Member

@sugeeshC, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <john.doe@email.org> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

1 similar comment
@vmwclabot
Copy link
Member

@sugeeshC, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <john.doe@email.org> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

@sugeeshC sugeeshC merged commit 286f4b8 into main Jul 10, 2024
15 checks passed
@VenelinBakalov VenelinBakalov deleted the fix/210-blueprint-folder-name-mismatch-issue branch July 29, 2024 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working version/patch The change is a non-breaking bugfix
Projects
None yet
6 participants