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.

Sign up for GitHub

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

Jump to bottom

IstioOperator Helm chart for Kyma 2.0 #11702

Merged
merged 2 commits into from Aug 23, 2021
Merged

IstioOperator Helm chart for Kyma 2.0 #11702

merged 2 commits into from Aug 23, 2021

Conversation

cnvergence
Copy link
Contributor

@cnvergence cnvergence commented Jul 16, 2021 •
edited

Description

Changes proposed in this pull request:

  • Implement new istio chart containing minimal IstioOperator CR including only changes which kyma is introducing
  • Support for profiles: production and evaluation

Related issue(s)

Fixes #11636

@kyma-bot
Copy link
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@kyma-bot kyma-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Jul 16, 2021
@cnvergence cnvergence force-pushed the istio-operator-helm-chart branch 2 times, most recently from 9e6c498 to 2946a05 Compare July 20, 2021 13:29
@netlify
Copy link

netlify bot commented Jul 20, 2021

✔️ 🥰 Documentation preview ready! 🥰

🔨 Explore the source changes: 9e6c498

🔍 Inspect the deploy log: https://app.netlify.com/sites/kyma-project-docs-preview/deploys/60f6cee771e40800080676fe

😎 Browse the preview: https://deploy-preview-11702--kyma-project-docs-preview.netlify.app

@netlify
Copy link

netlify bot commented Jul 20, 2021 •
edited

✔️ 🥰 Documentation preview ready! 🥰

🔨 Explore the source changes: 2c4df98

🔍 Inspect the deploy log: https://app.netlify.com/sites/kyma-project-docs-preview/deploys/612368a57f061b0007b01eac

😎 Browse the preview: https://deploy-preview-11702--kyma-project-docs-preview.netlify.app

@cnvergence cnvergence marked this pull request as ready for review August 6, 2021 12:40
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 6, 2021
@cnvergence cnvergence requested a review from strekm August 6, 2021 12:40
@cnvergence cnvergence added the area/service-mesh Issues or PRs related to service-mesh label Aug 6, 2021
@cnvergence
Copy link
Contributor Author

/retest

mjakobczyk
mjakobczyk previously approved these changes Aug 16, 2021
View reviewed changes
@kyma-bot kyma-bot added the lgtm Looks good to me! label Aug 16, 2021
@cnvergence
Copy link
Contributor Author

/test pre-main-kyma-integration-k3d

@cnvergence
Copy link
Contributor Author

/test pre-main-kyma-gke-integration

2 similar comments
@cnvergence
Copy link
Contributor Author

/test pre-main-kyma-gke-integration

@Tomasz-Smelcerz-SAP
Copy link
Member

/test pre-main-kyma-gke-integration

@kyma-bot kyma-bot removed the lgtm Looks good to me! label Aug 23, 2021
@kyma-bot
Copy link
Contributor

@cnvergence: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Rerun command
pre-main-cluster-users-integration-minikube 29bf932 link /test pre-main-cluster-users-integration-minikube
pre-main-kyma-gardener-gcp-eventing 2946a05 link /test pre-main-kyma-gardener-gcp-eventing
pre-main-kyma-gardener-gcp-api-gateway 2946a05 link /test pre-main-kyma-gardener-gcp-api-gateway
pre-main-check-users-map 2c4df98 link /test pre-main-check-users-map

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kyma-bot kyma-bot added the lgtm Looks good to me! label Aug 23, 2021
@kyma-bot kyma-bot merged commit fde63e2 into kyma-project:main Aug 23, 2021
@cnvergence cnvergence mentioned this pull request Aug 23, 2021
Closed
@mjakobczyk mjakobczyk mentioned this pull request Aug 30, 2021
Merged
@cnvergence cnvergence deleted the istio-operator-helm-chart branch September 8, 2021 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pbochynski pbochynski pbochynski approved these changes

@mjakobczyk mjakobczyk mjakobczyk left review comments

@a-thaler a-thaler Awaiting requested review from a-thaler

@PK85 PK85 Awaiting requested review from PK85

@strekm strekm Awaiting requested review from strekm

@Tomasz-Smelcerz-SAP Tomasz-Smelcerz-SAP Awaiting requested review from Tomasz-Smelcerz-SAP

@piotrkpc piotrkpc Awaiting requested review from piotrkpc

@werdes72 werdes72 Awaiting requested review from werdes72

@mmitoraj mmitoraj Awaiting requested review from mmitoraj

Assignees

@pbochynski pbochynski

@mjakobczyk mjakobczyk

Labels
area/service-mesh Issues or PRs related to service-mesh lgtm Looks good to me! size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Helm chart for istio istiooperator file
5 participants
@cnvergence @kyma-bot @Tomasz-Smelcerz-SAP @pbochynski @mjakobczyk