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

[TASK] Make use of the gatewayAddress property instead of the deprecated brokerContactPoint in Operate #1646

Open
2 tasks
romansmirnov opened this issue Apr 18, 2024 · 2 comments · May be fixed by #1647
Open
2 tasks

Comments

@romansmirnov
Copy link
Member

Related to:

With camunda/camunda#17062, Operate is distributed together with Zeebe as a Single JAR (Distribution). By that, the Operate configuration is part of the application.yaml file and no longer in application.yml (without an a in yaml). Additionally, a while ago the configuration property brokerContactPoint was marked as deprecated, and instead the property gatewayAddress must be used.

All that together, it may cause Operate to try to connect to Zeebe via localhost:26500, which is caused by the order Spring loads/applies the application.yml and application.yaml files. Plus, how Operate sets the gatewayAddress (even when using the brokerContactPoint), see camunda/camunda#17571

While this will be fixed in Operate itself, I suggest adjusting the config map accordingly to reflect the changes in the Helm Charts as well.

Overview:

  • In Operate, use gatewayAddress instead of brokerContactPoint
  • In Operate, change application.yml to application.yaml to be mounted

Actions:

  • This change must be released with 8.6.0.
@romansmirnov
Copy link
Member Author

Opened a PR to apply the changes: #1647

@aabouzaid aabouzaid added this to the 8.6 Release Cycle milestone Apr 23, 2024
@aabouzaid
Copy link
Member

@romansmirnov Thanks for the issue and the PR.
Should it be part of the 8.6 "cycle" (before October) or the 8.6 "release" (in October)?

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

Successfully merging a pull request may close this issue.

2 participants