Repository navigation
Bumps components - #2546
Bumps components#2546
Conversation
The generic backbeat.configurationOverrides map reaches backbeat as environment variables, and 9.5 answers most of those names from an entrypoint that rewrites conf/config.json in place. The operator mounts that directory read only, so the container dies before it serves anything: KAFKA_BACKLOG_METRICS_INTERVALS is one of the 57 names rewritten that way, which is what failed this suite eight runs in a row. 9.6 deleted the entrypoint and reads those names from the configuration schema itself (BB-808), where joi also coerces the value, so the interval arrives as the number the schema declares rather than as a string. That the operator passes them to a backbeat that cannot take them is ZKOP-605. Issue: ZENKO-5359
backlogMetricsIntervalSeconds was a dedicated CR field, added by zenko-operator#626 and reverted by #633 right after v1.8.16 in favour of the generic backbeat.configurationOverrides map. The two spellings never coexist in a released operator: v1.8.16 knows only the field, and every 1.8 release after it knows only the override. So the e2e config and the operator pin have to move together, and bumping the operator without this change breaks the whole suite on a rejected field. KAFKA_BACKLOG_METRICS_INTERVALS is backbeat's declared env alias for kafka.backlogMetrics.intervalS, so the value reaches the same config key with the same unit the removed field used -- as of 9.6, bumped just before this, which is also the first release the map can be passed to at all. It is set on the four consumers whose lag the lifecycle conductor waits on. Issue: ZENKO-5359
Issue: ZENKO-5370
Issue: ZENKO-5370
Issue: ZENKO-5370
Issue: ZENKO-5370
Hello francoisferrand,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Incorrect fix versionThe
Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:
Please check the |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
|
5f2e14d to
0033600
Compare
…configuration-overrides' into improvement/ZENKO-5370
0033600 to
5bcfd6d
Compare
|
/approve |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
This pull request did not target the following hotfix branch(es) so they
Please check the status of the associated issue ZENKO-5370. Goodbye francoisferrand. The following options are set: approve |
Issue: ZENKO-5370