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

Change to go full knative with the operator #375

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gabriel-farache
Copy link
Collaborator

Closes https://issues.redhat.com/browse/FLPATH-1406
Change to go full knative with the operator

@gabriel-farache
Copy link
Collaborator Author

/hold until apache/incubator-kie-kogito-serverless-operator#467 is merged and updated in the orchestrator helm chart repo

@gabriel-farache
Copy link
Collaborator Author

@rgolangh why was this PR closed?

@rgolangh
Copy link
Collaborator

my bad, reopening

@rgolangh rgolangh reopened this Jul 18, 2024
@rgolangh
Copy link
Collaborator

why is this move2kube specific and not system wide instructions?

charts/move2kube/templates/03-knative-resources.yaml Outdated Show resolved Hide resolved
charts/move2kube/values.yaml Outdated Show resolved Hide resolved
@gabriel-farache
Copy link
Collaborator Author

gabriel-farache commented Jul 18, 2024

why is this move2kube specific and not system wide instructions?

@rgolangh because currently, only M2K workflow is using a broker and we the new changes, we won't have to set the K_SINK manually after the deployment; it will be done by the SinkBinding created by the operator

The other change in the README is to only set necessary scc instead of the high level anyuuid

@gabriel-farache
Copy link
Collaborator Author

the CI will fail until orchestrator-helm-chart is updated with OSL 1.35 to support eventing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants