opa icon indicating copy to clipboard operation
opa copied to clipboard

Issue-6559 Add ability to create decision labels within a policy

Open tsidebottom opened this issue 1 year ago • 3 comments

Why the changes in this PR are needed?

Adds a DecisionLabel Field (a Map Object) to the Decision Log output to track Policy Result Data.

What are the changes in this PR?

  • A new field (DecisionLabel) was added to the Decision Log
  • This field was propagated backwards through the Policy Evaluation process
    • includes updates to files in Topdown, Rego, and SDK domains
  • A new builtin (DecisionLabelAdd) was created to populate the new field during evaluation

Notes to assist PR review:

Further comments:

Addresses #6559

tsidebottom avatar Apr 05 '24 21:04 tsidebottom

Deploy Preview for openpolicyagent ready!

Name Link
Latest commit 4070f53a27381e29c46e85816a548781c1668f18
Latest deploy log https://app.netlify.com/sites/openpolicyagent/deploys/6621259b68b7f400085fb353
Deploy Preview https://deploy-preview-6681--openpolicyagent.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Apr 05 '24 21:04 netlify[bot]

Thanks for the contribution @tsidebottom. FYI we've added some comments on https://github.com/open-policy-agent/opa/issues/6559 about the proposed feature including the latest one here. It would be helpful to identify alternate ways of achieving what's needed for https://github.com/open-policy-agent/opa/issues/6559. Thanks.

ashutosh-narkar avatar Apr 08 '24 18:04 ashutosh-narkar

This pull request has been automatically marked as stale because it has not had any activity in the last 30 days.

stale[bot] avatar May 18 '24 23:05 stale[bot]

Closing this PR for now. We can continue the discussion in https://github.com/open-policy-agent/opa/issues/6559. There are other approaches discussed in the issue on how to address this. In its current form this PR is unlikely to be merged and there's been no activity on it for a while so feel free to re-open in draft mode if needed with modifications. Thanks for your time and work on this!

ashutosh-narkar avatar Jun 05 '24 17:06 ashutosh-narkar