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

set resource ownership when generating JWS domain #2746

Merged
merged 1 commit into from
Sep 29, 2024

Conversation

chandrasekhar1996
Copy link
Contributor

Description

While generating JWS domain object, the domain resource ownership field was missing. This led to an issue when using zms-cli reset-domain-resource-ownership domain command since we check if resource ownership is already empty or not

if domainType && domainData.ResourceOwnership != nil {

Contribution Checklist:

  • The pull request does not introduce any breaking changes
  • I have read the contribution guidelines.
  • Create an issue and link to the pull request.

Attach Screenshots (Optional)

Signed-off-by: craman <chandrasekhar.raman@yahooinc.com>
@havetisyan havetisyan merged commit ec3530a into AthenZ:master Sep 29, 2024
2 checks passed
@chandrasekhar1996 chandrasekhar1996 deleted the jwsDomain branch September 29, 2024 18:04
havetisyan added a commit that referenced this pull request Sep 30, 2024
…from master to the jett-12 branch

Signed-off-by: Henry Avetisyan <hga@yahooinc.com>
havetisyan added a commit that referenced this pull request Sep 30, 2024
…from master to the jett-12 branch (#2749)

Signed-off-by: Henry Avetisyan <hga@yahooinc.com>
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.

2 participants