-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Opa blog post #15134
Opa blog post #15134
Conversation
Signed-off-by: Charlie Egan <[email protected]>
Add intro, adjust policy
Skipping CI for Draft Pull Request. |
😊 Welcome! This is either your first contribution to the Istio documentation repo, or
Thanks for contributing! Courtesy of your friendly welcome wagon. |
/retest |
Thanks for the submission, @antonioberben! Some thoughts before diving into a full edit:
Let me know how you'd like to proceed and I can either help get this into shape for a doc page or a revised blog post. |
Hi @antonioberben Nice blog - does this instruction work with ambient as well? |
Hi @linsun , it does not work in ambient. I could not make it work with it |
Tests are passing. Now awaiting @antonioberben to test the revised post top to bottom and take in @dhawton's fixes. |
Co-authored-by: Daniel Hawton <[email protected]>
Co-authored-by: Daniel Hawton <[email protected]>
Co-authored-by: Daniel Hawton <[email protected]>
Have you actually verified my changes? I offer no guarantee they work at
all.
|
Yes, it is tested from top to bottom. If @dhawton can run it quickly, then we can finish it today |
I've reviewed the instructions and the rendered post this morning, lgtm 👍 |
@craigbox , @dhawton , do you need anything else to approve this? @charlieegan3 has already tested it too. Thanks |
that'll do me |
In response to a cherrypick label: #15134 failed to apply on top of branch "release-1.23":
|
In response to a cherrypick label: new issue created for failed cherrypick: #15798 |
* add blog post * Add intro, adjust policy Signed-off-by: Charlie Egan <[email protected]> * Remove other titles and descriptions * Pass lint * sort alphabetically * sort alphabetically * Fix lint complains * Fix spelling merge * Sort spelling file * Fix linter claims * Fix publishing date * Update content/en/docs/ops/integrations/opa/index.md Co-authored-by: Charlie Egan <[email protected]> * Update content/en/docs/ops/integrations/opa/index.md Co-authored-by: Charlie Egan <[email protected]> * Update content/en/docs/ops/integrations/opa/index.md Co-authored-by: Charlie Egan <[email protected]> * Update content/en/docs/ops/integrations/opa/index.md Co-authored-by: Charlie Egan <[email protected]> * Update content/en/docs/ops/integrations/opa/index.md Co-authored-by: Charlie Egan <[email protected]> * Update content/en/docs/ops/integrations/opa/index.md Co-authored-by: Charlie Egan <[email protected]> * Fix review * fix linter * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Craig Box <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update .spelling Co-authored-by: Daniel Hawton <[email protected]> * Update .spelling Co-authored-by: Daniel Hawton <[email protected]> * Update .spelling Co-authored-by: Daniel Hawton <[email protected]> * Update .spelling Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Inline examples * remove doc * small lint fixes * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update content/en/blog/2024/l7-policy-with-opa/index.md Co-authored-by: Daniel Hawton <[email protected]> * Update title and description * Fix demo * Fix code block issue --------- Signed-off-by: Charlie Egan <[email protected]> Co-authored-by: Charlie Egan <[email protected]> Co-authored-by: Charlie Egan <[email protected]> Co-authored-by: Daniel Hawton <[email protected]> Co-authored-by: Craig Box <[email protected]>
Will go live when #15799 is approved |
* add blog post * Add intro, adjust policy * Remove other titles and descriptions * Pass lint * sort alphabetically * sort alphabetically * Fix lint complains * Fix spelling merge * Sort spelling file * Fix linter claims * Fix publishing date * Update content/en/docs/ops/integrations/opa/index.md * Update content/en/docs/ops/integrations/opa/index.md * Update content/en/docs/ops/integrations/opa/index.md * Update content/en/docs/ops/integrations/opa/index.md * Update content/en/docs/ops/integrations/opa/index.md * Update content/en/docs/ops/integrations/opa/index.md * Fix review * fix linter * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update .spelling * Update .spelling * Update .spelling * Update .spelling * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Inline examples * remove doc * small lint fixes * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update content/en/blog/2024/l7-policy-with-opa/index.md * Update title and description * Fix demo * Fix code block issue --------- Signed-off-by: Charlie Egan <[email protected]> Co-authored-by: Antonio B. <[email protected]> Co-authored-by: Charlie Egan <[email protected]> Co-authored-by: Charlie Egan <[email protected]> Co-authored-by: Daniel Hawton <[email protected]>
Description
This PR adds a blog post about how to use OPA with Istio and the benefits of both.
Adding co-author to the loop: @charlieegan3
Reviewers