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

feat: add Common Labels Support #167

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

bagnaram
Copy link
Contributor

@bagnaram bagnaram commented Nov 22, 2024

Common Labels to Helm Chart

Description

Adds optional common labels support which gets added to all deployable Kube resources.

References

Review Checklist

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

If you haven't done so yet, we would appreciate it if you could star the OpenFGA repository. :)

@bagnaram bagnaram force-pushed the bagnaram/helm-common-labels branch from a8cd10d to b57f95e Compare November 22, 2024 15:57
@bagnaram bagnaram marked this pull request as ready for review November 22, 2024 15:58
@bagnaram bagnaram requested review from a team as code owners November 22, 2024 15:58
@bagnaram bagnaram changed the title WIP: Add Common Labels Support Add Common Labels Support Nov 22, 2024
@bagnaram bagnaram force-pushed the bagnaram/helm-common-labels branch from b57f95e to b9f7e47 Compare November 22, 2024 16:43
@hello-caleb
Copy link

hello-caleb commented Nov 22, 2024

Thank you for your contribution to the Helm charts! If you're comfortable with it, we encourage contributors to update documentation where it makes sense. In this case, you may want to mention commonLabels in the chart’s README.md and include a usage example in values.yaml to help users. If you do wish to update the documentation, you can do so in a separate PR and link it in this PRs Description.

@bagnaram bagnaram force-pushed the bagnaram/helm-common-labels branch from b9f7e47 to 8946980 Compare November 22, 2024 18:15
@bagnaram bagnaram requested review from a team as code owners November 22, 2024 18:15
@bagnaram bagnaram force-pushed the bagnaram/helm-common-labels branch 4 times, most recently from 5555196 to a5ab670 Compare November 22, 2024 18:24
@bagnaram
Copy link
Contributor Author

Thank you for reviewing @hello-caleb ! I am not sure which separate documentation I need to update but I have now provided usage examples.

@bagnaram bagnaram force-pushed the bagnaram/helm-common-labels branch from a5ab670 to d2570dc Compare November 22, 2024 18:28
Copy link
Member

@rhamzeh rhamzeh left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Added a small suggestion

charts/openfga/values.schema.json Outdated Show resolved Hide resolved
@rhamzeh rhamzeh changed the title Add Common Labels Support feat: add Common Labels Support Nov 22, 2024
@bagnaram bagnaram force-pushed the bagnaram/helm-common-labels branch from d2570dc to d34d642 Compare November 22, 2024 18:47
@bagnaram bagnaram requested review from rhamzeh and jalyng November 25, 2024 14:57
@bagnaram bagnaram force-pushed the bagnaram/helm-common-labels branch from d34d642 to 3e261f6 Compare November 27, 2024 14:01
rhamzeh
rhamzeh previously approved these changes Nov 27, 2024
@rhamzeh
Copy link
Member

rhamzeh commented Nov 27, 2024

Thank you @bagnaram :)

@rhamzeh rhamzeh merged commit deeb315 into openfga:main Nov 27, 2024
4 checks passed
@rhamzeh rhamzeh mentioned this pull request Dec 2, 2024
4 tasks
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.

4 participants