Skip to content

Add option to select worload using lables for authn policy.#755

Merged
istio-testing merged 2 commits intoistio:release-1.1from
diemtvu:authn-labels
Jan 16, 2019
Merged

Add option to select worload using lables for authn policy.#755
istio-testing merged 2 commits intoistio:release-1.1from
diemtvu:authn-labels

Conversation

@diemtvu
Copy link
Contributor

@diemtvu diemtvu commented Jan 11, 2019

This is a prep work to use label selector for authentication. Selector using service name will be converted to labels internally.

@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Jan 11, 2019
Copy link
Contributor

@costinm costinm left a comment

Choose a reason for hiding this comment

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

/lgtm

@yangminzhu
Copy link
Contributor

Should this go to 1.1 branch or are you planning to merge this after master is unblocked?

@diemtvu diemtvu requested review from geeknoid and ozevren January 14, 2019 21:57
@ozevren
Copy link
Contributor

ozevren commented Jan 15, 2019

/lgtm

@diemtvu
Copy link
Contributor Author

diemtvu commented Jan 15, 2019

@hklai, can we have this merge to master to have some implementation started. Thanks.

@hklai
Copy link
Contributor

hklai commented Jan 15, 2019

@diemtvu perhaps not until it is unlocked. we don't want significant divergence between 1.1 and master.

@diemtvu diemtvu changed the base branch from master to release-1.1 January 16, 2019 18:47
@istio-testing
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: costinm, diemtvu, ozevren

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [costinm,diemtvu,ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@diemtvu
Copy link
Contributor Author

diemtvu commented Jan 16, 2019

/test api-presubmit

@istio-testing istio-testing merged commit aec9db9 into istio:release-1.1 Jan 16, 2019
hklai added a commit that referenced this pull request Jan 19, 2019
* assorted doc updates (#757)

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* add optional incremental flag to ResponseSink and ResourceSource services (#762)

* Add option to select worload using lables for authn policy. (#755)

* Add option to select workload using labels for authn policy

* Make proto-commit
@diemtvu diemtvu deleted the authn-labels branch April 15, 2019 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants