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

IAC-791 Full Support of the vRO Policies Feature #202

Conversation

akashayyappan
Copy link
Contributor

@akashayyappan akashayyappan commented Dec 8, 2023

Description

Checklist

  • Lint and unit tests pass locally with my changes
  • 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 (if appropriate)
  • I have updated CHANGELOG.md with a short summary of the changes introduced
  • I have tested against live environment, if applicable
  • I have added relevant usage information (As-built)
  • Any structure and/or content vRA-NG improvements are synchronized with vra-ng and ts-vra-ng archetypes
  • Every new or updated Installer property is documented in docs/archive/doc/markdown/use-bundle-installer.md
  • Dependencies in pom.xml are up-to-date
  • My changes have been rebased and squashed to the minimal number of relevant commits
  • My changes have a descriptive commit message with a short title, including a Fixed #XXX - or Closed #XXX - prefix to auto-close the issue

Testing

Tested the changes with a live enivronment with posibile scenarios.

Release Notes

Implemeneted functionality to support full vRO Policy Feature

Related issues and PRs

Updated the Samples repo files as per the new implementation.
Sample Repo PR: PR-1

@akashayyappan akashayyappan requested a review from a team as a code owner December 8, 2023 08:26
@Michaelpalacce
Copy link
Collaborator

Seems like the build is failing, lets fix it

@akashayyappan
Copy link
Contributor Author

Seems like the build is failing, lets fix it

The failure is due to changes in the interface parameters which fails the samples repo build. It is decided to implement versioning to be backward compatible. I'm currently working on it, will update the PR along with the changes mentioned in the comments.

@joroaf
Copy link
Collaborator

joroaf commented Dec 12, 2023

@akashayyappan let's introduce versioning of the interface and keep backward compatibility.

@Michaelpalacce
Copy link
Collaborator

One more content from me regarding the limitation of the element types, also sync with master

@VenelinBakalov
Copy link
Collaborator

@Michaelpalacce I see there is 1 change requested by you, is it still valid or we can close the PR?

@VenelinBakalov
Copy link
Collaborator

I am linking this PR to close issue #77
@Michaelpalacce @joroaf ping me if I am wrong

@akashayyappan
Copy link
Contributor Author

akashayyappan commented Dec 18, 2023

I am linking this PR to close issue #77 @Michaelpalacce @joroaf ping me if I am wrong

This PR is for vRO Policies. You can check this task: IAC-791.

@VenelinBakalov
Copy link
Collaborator

I see, thanks, link removed

@vmwclabot
Copy link
Member

@akashayyappan, 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

@akashayyappan, 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.

@akashayyappan akashayyappan force-pushed the IAC-791-Full-support-of-the-vRO-Policies-feature branch from fd7bb8c to 2abf712 Compare December 19, 2023 04:58
@vmwclabot
Copy link
Member

@akashayyappan, 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

@akashayyappan, 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.

@akashayyappan akashayyappan force-pushed the IAC-791-Full-support-of-the-vRO-Policies-feature branch from 2abf712 to d784a87 Compare December 19, 2023 05:06
@vmwclabot
Copy link
Member

@akashayyappan, 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

@akashayyappan, 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.

@akashayyappan akashayyappan force-pushed the IAC-791-Full-support-of-the-vRO-Policies-feature branch from 3d854cb to 2e06699 Compare December 19, 2023 05:11
@VenelinBakalov
Copy link
Collaborator

Oks, thanks. I see a minor merge conflict in the Changelog, could you check this, in the meantime I will check with @Michaelpalacce

@krasimirr krasimirr linked an issue Jan 26, 2024 that may be closed by this pull request
Copy link
Collaborator

@Michaelpalacce Michaelpalacce left a comment

Choose a reason for hiding this comment

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

2 small changes, otherwise look good

vro-types/vrotsc-annotations/index.d.ts Show resolved Hide resolved
## Policy Template Versioning
Policy template now has two versions for backward compatability. Template versions can be changed using the **templateVersion** attribute. Use v2 for full support of creating Policy with variables and multiple elements.

**NOTE:** templateVersion attribute is optional. if not provided, templateVersion will be considered as v1 by default.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't see this being used in any way, we've defined it but nothing else, how are we validating this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Policy Template Versioning was introduced to make the new changes be backward compatible. If user provides version as v2 which is optional, user can create policy with full support.

@vmwclabot
Copy link
Member

@akashayyappan, 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

@akashayyappan, 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.

@vmwclabot
Copy link
Member

@akashayyappan, 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.

3 similar comments
@vmwclabot
Copy link
Member

@akashayyappan, 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.

@vmwclabot
Copy link
Member

@akashayyappan, 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.

@vmwclabot
Copy link
Member

@akashayyappan, 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.

Copy link

github-actions bot commented Mar 7, 2024

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale label Mar 7, 2024
@VenelinBakalov
Copy link
Collaborator

@Michaelpalacce could you double check, I think there is a change request on your side, is it still valid?

@github-actions github-actions bot removed the Stale label Mar 8, 2024
@unbreakabl3
Copy link
Contributor

Hi, @Michaelpalacce .
Is it possible somehow to fix the merge conflict to start to use it?

@vmwclabot
Copy link
Member

@akashayyappan, 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

@akashayyappan, 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.

@vmwclabot
Copy link
Member

@akashayyappan, 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

@akashayyappan, 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.

@akashayyappan akashayyappan force-pushed the IAC-791-Full-support-of-the-vRO-Policies-feature branch from a02654c to 0866746 Compare April 22, 2024 06:57
Signed-off-by: Akash Ayyappan <ayyappana@vmware.com>
@akashayyappan akashayyappan force-pushed the IAC-791-Full-support-of-the-vRO-Policies-feature branch from 0866746 to 80aa1d6 Compare April 22, 2024 07:28
@ynedelchev ynedelchev merged commit 775220f into vmware:main Apr 22, 2024
6 checks passed
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.

8 participants