-
Notifications
You must be signed in to change notification settings - Fork 103
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
Add OPA support for Policy Set API's #575
Conversation
👋 Hi @mrinalirao ! Thank you for your contribution! It looks like this PR might be targeting a feature that is in beta. We have guidance in our docs about how to add tests for beta features, plus how to comment when a feature is in beta. Could you please update the PR based on that guidance? 💖 I'll take a more detailed look and add more thoughts ✨ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One typo change but should not block this PR.
Almost forgot. You'll need a Changelog entry for this. |
# Conflicts: # CHANGELOG.md
Reminder to the contributor that merged this PR: if your changes have added important functionality or fixed a relevant bug, open a follow-up PR to update CHANGELOG.md with a note on your changes. |
Description
Add OPA support to the go-tfe client for policy sets. OPA is still in beta phase.
This PR only contains changes to the policy set API's.
For a complete list of API's that will be updated in subsequent PRs see : https://github.com/hashicorp/terraform-docs-common/pull/141/files
Api's updated:
Testing plan
Integration tests added
External links
Output from tests
Needs the following env vars:
TFE_ADDRESS
TFE_TOKEN
OAUTH_CLIENT_GITHUB_TOKEN
GITHUB_POLICY_SET_IDENTIFIER
Set ENABLE_BETA=1