Missing config for mattermost-lts #693

Closed
opened 2025-10-07 19:49:59 +00:00 by decentral1se · 2 comments
Owner

Similar #692 when me and @knoflook hung out and found abra bugs.

  • mattermost-lts
  • version 2.0.5... something
  • deploy overview is missing the conf version
Similar https://git.coopcloud.tech/toolshed/abra/issues/692 when me and @knoflook hung out and found `abra` bugs. * `mattermost-lts` * version 2.0.5... something * deploy overview is missing the conf version
decentral1se added the
bug
label 2025-10-07 19:49:59 +00:00
decentral1se added this to the Abra v0.12 project 2025-10-07 19:49:59 +00:00
Owner

it looks for the version key like this: versionKey := strings.ToUpper(configName) + "_VERSION"
https://git.coopcloud.tech/toolshed/abra/src/branch/main/pkg/deploy/utils.go#L241
config name is abra_mattermost_entrypoint in compose.yml but it's using ENTRYPOINT_VERSION as the variable to set the version. and so that's exported in abra.sh

so abra instead of looking for the variable that is part of the name of the config (in our case ENTRYPOINT_VERSION) in abra.sh looks for a variable named the same as the config section (in our case abra_mattermost_entrypoint but capitalised and with _VERSION at the end. But that doesn't exist.

it looks for the version key like this: `versionKey := strings.ToUpper(configName) + "_VERSION"` https://git.coopcloud.tech/toolshed/abra/src/branch/main/pkg/deploy/utils.go#L241 config name is `abra_mattermost_entrypoint` in [`compose.yml`](https://git.coopcloud.tech/coop-cloud/mattermost-lts/src/branch/main/compose.yml#L69) but it's using `ENTRYPOINT_VERSION` as the variable to set the version. and so that's exported in [`abra.sh`](https://git.coopcloud.tech/coop-cloud/mattermost-lts/src/branch/main/abra.sh) so abra instead of looking for the variable that is part of the name of the config (in our case `ENTRYPOINT_VERSION`) in `abra.sh` looks for a variable named the same as the config section (in our case `abra_mattermost_entrypoint` but capitalised and with `_VERSION` at the end. But that doesn't exist.
decentral1se moved this to Prioritised in Abra v0.12 on 2025-10-15 08:03:35 +00:00
decentral1se moved this to Backlog in Abra v0.12 on 2025-10-17 17:31:34 +00:00
decentral1se moved this to Prioritised in Abra v0.12 on 2025-10-17 17:31:50 +00:00
Author
Owner

@knoflook not the answer you're probably looking for but this is actually the intended behaviour. See toolshed/organising#680 and the linked issues for hot takes, chaos commits and all the rest. It's documented now: https://docs.coopcloud.tech/maintainers/handbook/#manage-configs From what I can tell, the mattermost-lts recipe has been updated.

@knoflook not the answer you're probably looking for but this is actually the intended behaviour. See https://git.coopcloud.tech/toolshed/organising/issues/680 and the linked issues for hot takes, chaos commits and all the rest. It's documented now: https://docs.coopcloud.tech/maintainers/handbook/#manage-configs From what I can tell, the [`mattermost-lts`](https://git.coopcloud.tech/coop-cloud/mattermost-lts) recipe has been updated.
decentral1se moved this to Done in Abra v0.12 on 2025-11-04 14:10:56 +00:00
Sign in to join this conversation.
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: toolshed/abra#693
No description provided.