Skip to content

[19.0][REF] cloud_platform,monitoring_prometheus: server_environment should not be a required dependency#522

Merged
ivantodorovich merged 2 commits into
19.0from
19-remove-server-env-dep
May 20, 2026
Merged

[19.0][REF] cloud_platform,monitoring_prometheus: server_environment should not be a required dependency#522
ivantodorovich merged 2 commits into
19.0from
19-remove-server-env-dep

Conversation

@yankinmax
Copy link
Copy Markdown
Member

PR is aimed to remove server_environment dependency from cloud_platform and monitoring_prometheus in the scope of this issue:

https://camptocamp.atlassian.net/browse/BSRD-887

Forward-port of:

@yankinmax
Copy link
Copy Markdown
Member Author

This should be checked after the 19.0 branch is activated (modules were "installable": False).

Copy link
Copy Markdown
Member

@ivantodorovich ivantodorovich left a comment

Choose a reason for hiding this comment

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

Just forward port the commit, but please do not change installable to True, nor update the manifest version.

These modules are not migrated so we don't need to change that here.

Comment thread monitoring_prometheus/__manifest__.py Outdated
Comment thread monitoring_prometheus/__manifest__.py Outdated
Comment thread cloud_platform/__manifest__.py Outdated
@yankinmax yankinmax force-pushed the 19-remove-server-env-dep branch 2 times, most recently from a131b2c to 1d0199b Compare March 31, 2026 15:17
Comment thread cloud_platform/__manifest__.py Outdated
Comment thread monitoring_prometheus/__manifest__.py Outdated
@yankinmax yankinmax force-pushed the 19-remove-server-env-dep branch from 1d0199b to dc4693d Compare April 2, 2026 08:32
@yankinmax yankinmax force-pushed the 19-remove-server-env-dep branch from dc4693d to f68b19b Compare May 20, 2026 06:57
@ivantodorovich ivantodorovich merged commit 2df49ee into 19.0 May 20, 2026
4 checks passed
@ivantodorovich ivantodorovich deleted the 19-remove-server-env-dep branch May 20, 2026 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants