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

Modify subscription controller chart #3007

Merged

Conversation

suleymanakbas91
Copy link
Contributor

Description

Changes proposed in this pull request:

  • Modify subscription controller chart
  • Delete test resources as we have a chart now

Related issue(s)

See #2301

@ghost
Copy link

ghost commented Mar 6, 2019

Please restrict the access rights as much as possible, I suppose that the following rights are enough

  • for KN channels:
    verbs: ["get", "create", "update","delete"]

  • for KN subscription:
    verbs: ["get", "create", "delete"]

@suleymanakbas91 suleymanakbas91 merged commit 1ac4f16 into kyma-project:master Mar 7, 2019
@suleymanakbas91 suleymanakbas91 deleted the subscription-controller branch March 7, 2019 07:33
Demonsthere pushed a commit to Demonsthere/kyma that referenced this pull request Mar 7, 2019
* Modify subscription controller chart
rakesh-garimella added a commit to rakesh-garimella/kyma that referenced this pull request Mar 7, 2019
* master:
  Modify subscription controller chart (kyma-project#3007)
  Adding the ca-certificates file to the scratch image (kyma-project#3029)
  Rename UI-API layer imports (kyma-project#2973)
  Use custom image in kyma integration tests job (kyma-project#3009)
  Append kyma subscription namespace to knative subscription name (kyma-project#3008)
  Update event-bus and event-service versions (kyma-project#3006)
  Introduce naming of the gophers component (kyma-project#2901)
  Update Connector Service chart and READMEs (kyma-project#2796)
  Extract nginx to a separate chart (kyma-project#2812)
  Upgrade to k8s version 1.12 (kyma-project#2976)
  Application Gateway renaming - bump (kyma-project#2933)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/eventing Issues or PRs related to eventing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants