Skip to content

update topaz config new#742

Merged
aserto-bot merged 6 commits into
mainfrom
upd/topaz-config-new
Jul 11, 2026
Merged

update topaz config new#742
aserto-bot merged 6 commits into
mainfrom
upd/topaz-config-new

Conversation

@gertd

@gertd gertd commented Jul 11, 2026

Copy link
Copy Markdown
Member

Update the topaz config new command implementation

  • update config references to 'aserto'
  • unfify the use of the --force short to -f across commands
  • extract the OPA template payloads

@gertd gertd marked this pull request as ready for review July 11, 2026 19:11
@coveralls

coveralls commented Jul 11, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 29165598880

Coverage remained the same at 7.206%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 10339
Covered Lines: 745
Line Coverage: 7.21%
Coverage Strength: 0.08 hits per line

💛 - Coveralls

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the topaz config new-related configuration surface by removing hard-coded Aserto UI origins from example/config templates, standardizing CLI flag short-hands (notably -f for --force), and extracting/reusing OPA template payload fragments (including plugin blocks).

Changes:

  • Remove *.aserto.com / *aserto-console.netlify.app from allowed_origins in templates, schema examples, test assets, and docs.
  • Standardize --force short flag usage to -f across commands touched in this PR.
  • Extract OPA template fragments and add/standardize plugin configuration blocks (decision logger + aserto_edge) in example configs.

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
topazd/tests/assets/config/peoplefinder.yaml Removes Aserto origins and adds OPA plugins config block.
topazd/tests/assets/config/config-with-tls.yaml Removes Aserto origins and adds OPA plugins config block.
topazd/tests/assets/config/config-no-tls.yaml Removes Aserto origins and adds OPA plugins config block.
topazd/authorizer/plugins/edge/aserto_edge.md Adds documentation for the aserto_edge OPA plugin configuration.
topaz/cmd/directory/manifest.go Adds -f short flag for delete-manifest force option.
topaz/cmd/configure/new.go Normalizes CLI tags/flags for topaz config new command options.
topaz/cmd/certs/generate.go Adds -f short flag for cert generation force option.
pkg/config/templates.go Refactors config templates by extracting OPA template fragments and appending plugin blocks.
pkg/config/schema/config.yaml Updates schema/example config snippets: removes origins and adds plugin sections.
docs/examples/config-public-ghcr.yaml Updates example allowed_origins away from Aserto domains.
docs/examples/config-private-apcr.yaml Removes outdated private APCR example.
docs/examples/config-local-image.yaml Updates example allowed_origins away from Aserto domains.
docs/deployments/sidecar-deployment/topaz-configmap.yaml Removes Aserto origins from deployment configmap example.
docs/deployments/docker-compose/config/local.yaml Removes Aserto origins from docker-compose local config example.
docs/config.md Updates documentation examples: origins removal + adds/expands OPA service/plugin examples.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pkg/config/schema/config.yaml Outdated
Comment thread topaz/cmd/directory/manifest.go Outdated
Comment thread topazd/authorizer/plugins/edge/aserto_edge.md
Comment thread topazd/authorizer/plugins/edge/aserto_edge.md
@aserto-bot aserto-bot merged commit 4f06cb7 into main Jul 11, 2026
6 checks passed
@aserto-bot aserto-bot deleted the upd/topaz-config-new branch July 11, 2026 19:43
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