Skip to content
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

Receptor Access Control #377

Closed
fosterseth opened this issue Aug 10, 2021 · 4 comments
Closed

Receptor Access Control #377

fosterseth opened this issue Aug 10, 2021 · 4 comments
Assignees
Labels

Comments

@fosterseth
Copy link
Member

fosterseth commented Aug 10, 2021

Adds work submission authority verification to receptor

  • Control "role" can submit work on execution nodes or other control nodes
  • Execution "role" cannot submit work anywhere

Need the firewall feature to prevent execution nodes from connecting to control nodes' control service, in addition to the sign work request feature

Acceptance criteria:

  • Log in to execution node and attempt to submit work on another node on the mesh -- Work should fail / be revoked
  • Log in to control node and attempt to submit work to another control node, or an execution node -- Work should succeed
  • Log in to control node whose signed token has expired and attempt to submit work anywhere -- Work should fail / be revoked

Time estimate: 1 Sprint

see https://hackmd.io/@shanemcd/SJPn85FJt

@yagomarques
Copy link

yagomarques commented Oct 18, 2021

we've already tested this functionality on the PR and Testathon, it's working in basic topologies. Maybe we would like to test in more complex topologies.

@Zokormazo @thenets @CFSNM @jneedle

@jneedle jneedle assigned Zokormazo and thenets and unassigned yagomarques Oct 22, 2021
@jneedle
Copy link

jneedle commented Oct 22, 2021

@Zokormazo and @thenets tagging both of you to track this to closure. If we are confident with our current coverage, feel free to close.

@thenets
Copy link
Member

thenets commented Oct 22, 2021

@jneedle I think we don't have coverage for this on the AWX side yet. I couldn't find it, at least. After the fixtures we made two weeks ago, I believe that will not be too hard or consuming automate this test.

@Zokormazo
Copy link
Member

we have some level of coverage. Will open follow up item to track the rest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants