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

connect: various changes to make namespaces for intentions work more like for other subsystems #8194

Merged
merged 1 commit into from
Jun 26, 2020

Conversation

rboyer
Copy link
Member

@rboyer rboyer commented Jun 26, 2020

Highlights:

  • add new endpoint to query for intentions by exact match

  • using this endpoint from the CLI instead of the dump+filter approach

  • enforcing that OSS can only read/write intentions with a SourceNS or
    DestinationNS field of "default".

  • preexisting OSS intentions with now-invalid namespace fields will
    delete those intentions on initial election or for wildcard namespaces
    an attempt will be made to downgrade them to "default" unless one
    exists.

  • also allow the '-namespace' CLI arg on all of the intention subcommands

  • update lots of docs

…like for other subsystems

Highlights:

- add new endpoint to query for intentions by exact match

- using this endpoint from the CLI instead of the dump+filter approach

- enforcing that OSS can only read/write intentions with a SourceNS or
  DestinationNS field of "default".

- preexisting OSS intentions with now-invalid namespace fields will
  delete those intentions on initial election or for wildcard namespaces
  an attempt will be made to downgrade them to "default" unless one
  exists.

- also allow the '-namespace' CLI arg on all of the intention subcommands

- update lots of docs
@rboyer rboyer added the theme/connect Anything related to Consul Connect, Service Mesh, Side Car Proxies label Jun 26, 2020
@rboyer rboyer requested review from mkeeler and a team June 26, 2020 21:41
@rboyer rboyer self-assigned this Jun 26, 2020
Copy link
Member

@mkeeler mkeeler left a comment

Choose a reason for hiding this comment

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

:shipit:

@rboyer rboyer merged commit 462f0f3 into master Jun 26, 2020
@rboyer rboyer deleted the fix-namespaces-and-intentions branch June 26, 2020 21:59
mikemorris added a commit that referenced this pull request Sep 10, 2020
…rk more like for other subsystems (#8194)"

This reverts commit 462f0f3.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme/connect Anything related to Consul Connect, Service Mesh, Side Car Proxies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants