Skip to content

Disable policy deployment by default#20213

Merged
istio-testing merged 5 commits intoistio:masterfrom
howardjohn:mixer/disable-policy-by-default
Jan 18, 2020
Merged

Disable policy deployment by default#20213
istio-testing merged 5 commits intoistio:masterfrom
howardjohn:mixer/disable-policy-by-default

Conversation

@howardjohn
Copy link
Copy Markdown
Member

@howardjohn howardjohn commented Jan 15, 2020

No description provided.

@howardjohn howardjohn requested a review from a team as a code owner January 15, 2020 19:44
@howardjohn howardjohn requested a review from a team January 15, 2020 19:44
@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 15, 2020
@istio-testing istio-testing added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 15, 2020
@howardjohn
Copy link
Copy Markdown
Member Author

/retest

@howardjohn
Copy link
Copy Markdown
Member Author

Needed to add a new default "integ test" (as opposed to e2e/) default values since it hardcoded policy to enabled

@howardjohn howardjohn force-pushed the mixer/disable-policy-by-default branch from 332646a to b027e18 Compare January 16, 2020 17:11
@howardjohn howardjohn requested a review from a team as a code owner January 16, 2020 17:11
@howardjohn
Copy link
Copy Markdown
Member Author

/retest

@howardjohn howardjohn force-pushed the mixer/disable-policy-by-default branch from b027e18 to 3582562 Compare January 17, 2020 22:29
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assuming this controls the deployment itself, lgtm

@douglas-reid ?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, this is controls the deployment

@howardjohn
Copy link
Copy Markdown
Member Author

/retest

@howardjohn howardjohn force-pushed the mixer/disable-policy-by-default branch from 3582562 to 6e4f16f Compare January 18, 2020 00:55
@howardjohn howardjohn force-pushed the mixer/disable-policy-by-default branch from 6e4f16f to d03858e Compare January 18, 2020 02:49
Copy link
Copy Markdown
Contributor

@mandarjog mandarjog left a comment

Choose a reason for hiding this comment

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

Lgtm

@istio-testing istio-testing merged commit ccc1fa9 into istio:master Jan 18, 2020
howardjohn added a commit to howardjohn/istio that referenced this pull request Jan 19, 2020
Regression since istio#20213

We turned on policy, but disabled checks. Didn't notice since it only
runs in postsubmit.
istio-testing pushed a commit that referenced this pull request Jan 19, 2020
Regression since #20213

We turned on policy, but disabled checks. Didn't notice since it only
runs in postsubmit.
pull bot pushed a commit to lxmwq/istio that referenced this pull request Jan 19, 2020
Regression since istio#20213

We turned on policy, but disabled checks. Didn't notice since it only
runs in postsubmit.
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. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants