Skip to content

[FIX] utils: Normalize Odoo versions for semver validation#16

Merged
ruiztulio merged 2 commits into
OrchestSh:masterfrom
vauxoo-dev:master-fix_saasv-truiz
Jun 16, 2026
Merged

[FIX] utils: Normalize Odoo versions for semver validation#16
ruiztulio merged 2 commits into
OrchestSh:masterfrom
vauxoo-dev:master-fix_saasv-truiz

Conversation

@ruiztulio

Copy link
Copy Markdown
Contributor

The semantic versioning comparisons were failing when processing non-standard version formats such as 'saas-19.2' and 'master'. Because they didn't follow the strictly numerical format, they were evaluated as older/invalid versions.

This caused the entrypoint to incorrectly inject deprecated parameters like 'longpolling_port', 'xmlrpc_port', and 'logrotate' into the .odoorc when booting Odoo 19/20 instances.

This change:

  • Introduces NormalizeVersionForSemver to clean the version strings
  • Handles 'master' as v999.0 to always consider it the latest
  • Handles 'saas-X' prefixes by stripping them
  • Applies the normalizer in SetupWorker and SetDefaults logic

The semantic versioning comparisons were failing when processing
non-standard version formats such as 'saas-19.2' and 'master'.
Because they didn't follow the strictly numerical format, they were
evaluated as older/invalid versions.

This caused the entrypoint to incorrectly inject deprecated parameters
like 'longpolling_port', 'xmlrpc_port', and 'logrotate' into the .odoorc
when booting Odoo 19/20 instances.

This change:
- Introduces NormalizeVersionForSemver to clean the version strings
- Handles 'master' as v999.0 to always consider it the latest
- Handles 'saas-X' prefixes by stripping them
- Applies the normalizer in SetupWorker and SetDefaults logic
Pinned the Go version to '1.26' across all workflows (test, release, and lint)
to ensure consistent behavior and avoid compatibility issues.

Updated GitHub actions to their latest available major versions:
- actions/checkout to v6
- actions/setup-go to v6
- golangci/golangci-lint-action to v9
- goreleaser/goreleaser-action to v7
@ruiztulio ruiztulio force-pushed the master-fix_saasv-truiz branch from 2e94e69 to e1c066f Compare June 16, 2026 23:17
@ruiztulio ruiztulio merged commit 065b983 into OrchestSh:master Jun 16, 2026
3 checks passed
@ruiztulio ruiztulio deleted the master-fix_saasv-truiz branch June 16, 2026 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant