Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

istio-operator: Update to 1.7.3 #1086

Merged
merged 2 commits into from
Oct 16, 2020
Merged

istio-operator: Update to 1.7.3 #1086

merged 2 commits into from
Oct 16, 2020

Conversation

surajssd
Copy link
Member

@surajssd surajssd commented Oct 16, 2020

Release Notes: https://github.com/istio/istio/releases/tag/1.7.3

  • TODO: Add conversion tests.

@@ -1,8 +1,29 @@
hub: docker.io/istio
tag: 1.6.3
tag: 1.7.3
Copy link
Member Author

Choose a reason for hiding this comment

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

hub and tag were manually modified because istio is still shipping bad config in releases.

Labels: map[string]string{
"istio-operator-managed": "Reconcile",
"istio-injection": "disabled",
},
Copy link
Member Author

Choose a reason for hiding this comment

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

Istio added istio operator namespace config. And it was added with above labels.

invidian
invidian previously approved these changes Oct 16, 2020
Copy link
Member

@invidian invidian left a comment

Choose a reason for hiding this comment

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

LGTM

Release Notes: https://github.com/istio/istio/releases/tag/1.7.3

Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
This commit adds tests that verify if conversion of the `profile`
parameter happens correctly.

Another test is added to verify if the servicemonitor is created when
`enable_monitoring` is set to true.

Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
@surajssd surajssd merged commit 86d46aa into master Oct 16, 2020
@surajssd surajssd deleted the surajssd/update-istio branch October 16, 2020 12:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants