Skip to content

Backstage chart: fixing RHIDP/7806 in Orchestrator's SFP config - #173

Merged
rm3l merged 6 commits into
redhat-developer:mainfrom
elai-shalev:fix/7806
Jun 24, 2025
Merged

Backstage chart: fixing RHIDP/7806 in Orchestrator's SFP config#173
rm3l merged 6 commits into
redhat-developer:mainfrom
elai-shalev:fix/7806

Conversation

@elai-shalev

@elai-shalev elai-shalev commented Jun 19, 2025

Copy link
Copy Markdown

This PR will fix some issues addressed in RHIDP-7806. When using Orchestrator with an external db, environment variables were not passed to the job responsible to create the new 'sonataflow' database. This caused the SonataflowPlatform to be faulty, and the Job service and Data Index to fail.

This issue was fixed in this PR.

Checklist

  • For each Chart updated, version bumped in the corresponding Chart.yaml according to Semantic Versioning.
  • For each Chart updated, variables are documented in the values.yaml and added to the corresponding README.md. The pre-commit utility can be used to generate the necessary content. Use pre-commit run -a to apply changes. The pre-commit Workflow will do this automatically for you if needed.
  • JSON Schema template updated and re-generated the raw schema via the pre-commit hook.
  • Tests pass using the Chart Testing tool and the ct lint command.
  • If you updated the orchestrator-infra chart, make sure the versions of the Knative CRDs are aligned with the versions of the CRDs installed by the OpenShift Serverless operators declared in the values.yaml file. See Installing Knative Eventing and Knative Serving CRDs for more details.

@elai-shalev

Copy link
Copy Markdown
Author

@rm3l
Hey, this PR is for a bug fix for 1.7 in the backstage chart.

@rm3l

rm3l commented Jun 20, 2025

Copy link
Copy Markdown
Member

/cc

@openshift-ci
openshift-ci Bot requested a review from rm3l June 20, 2025 07:25
Comment thread charts/backstage/templates/sonataflows.yaml Outdated
Comment thread charts/backstage/templates/sonataflows.yaml Outdated
Comment thread charts/backstage/templates/sonataflows.yaml Outdated
Comment thread charts/backstage/templates/sonataflows.yaml Outdated
Comment thread charts/backstage/templates/sonataflows.yaml Outdated
Comment thread charts/backstage/values.yaml
Comment thread charts/backstage/README.md Outdated
Comment thread charts/backstage/templates/sonataflows.yaml Outdated
Comment thread charts/backstage/templates/sonataflows.yaml Outdated
@sonarqubecloud

Copy link
Copy Markdown

@rm3l rm3l left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Jun 24, 2025
@rm3l
rm3l merged commit 9ed0749 into redhat-developer:main Jun 24, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants