Skip to content

docs(proposals): add constraint routing proposal (#1271) - #34

Open
yehuditkerido wants to merge 1 commit into
praxis-proxy:mainfrom
yehuditkerido:proposal_01271
Open

yehuditkerido wants to merge 1 commit into
praxis-proxy:mainfrom
yehuditkerido:proposal_01271

Conversation

@yehuditkerido

Copy link
Copy Markdown

What does this PR do?

Add proposal 01271 to define the scope and motivation for enforcing PPE
candidate constraints during Praxis backend selection.

Related issue

praxis-proxy/praxis#1271

Checklist

  • Issue link is in proposal frontmatter
  • Authors and stakeholders listed
  • Repos list includes all affected repositories
  • Graduation criteria are specific and verifiable
  • First PR contains only What? and Why? (no How?)

Signed-off-by: Yehudit Kerido <ykerido@redhat.com>
@yehuditkerido
yehuditkerido requested a review from a team as a code owner October 29, 2026 08:02
@coderabbitai

coderabbitai Bot commented Oct 29, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Central YAML; Path: .coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: d5fc05bb-bed6-4a88-9c69-effea3cde374
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@szedan-rh

Copy link
Copy Markdown

@terylt - Could you please take look into the proposal ?

@terylt

terylt commented Nov 1, 2026

Copy link
Copy Markdown

Hi @yehuditkerido, I think this is a great piece to wire up in praxis and will provide a lot of benefits. The proposal is pretty high level but hopefully the related issue has enough information to make it a relatively easy integration.

One great addition after this integration would be adding it as a demo in https://github.com/praxis-proxy/demos/tree/main/demos. Maybe something around regions. It could either added apart of the existing policy engine broader demo: https://github.com/praxis-proxy/demos/tree/main/demos/policy-engine or a specialized one at the "demos" level like the ppe vault demo.

Let me know if you have any issues or need any help. I don't have maintainer rights on either this repo or the praxis repo so I can't assign you or review this PR formally.

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.

3 participants