Skip to content

ENT-12658: Inhibit management of share config.php file when mpf_disable_mission_portal_docroot_sync_from_share_gui is defined#2987

Merged
craigcomstock merged 2 commits into
cfengine:masterfrom
craigcomstock:ENT-12658/master
Feb 20, 2025
Merged

ENT-12658: Inhibit management of share config.php file when mpf_disable_mission_portal_docroot_sync_from_share_gui is defined#2987
craigcomstock merged 2 commits into
cfengine:masterfrom
craigcomstock:ENT-12658/master

Conversation

@craigcomstock

@craigcomstock craigcomstock commented Feb 12, 2025

Copy link
Copy Markdown
Contributor

In preparation for possibly removing the share/GUI folder entirely.

Ticket: ENT-12658
Changelog: title

…portal_docroot_sync_from_share_gui is defined

In preparation for possibly removing the share/GUI folder entirely.

Ticket: ENT-12658
Changelog: title
@craigcomstock

craigcomstock commented Feb 14, 2025

Copy link
Copy Markdown
Contributor Author

blocked by cfengine/cf-remote#124, that bug is what is breaking acceptance_tests

@craigcomstock craigcomstock marked this pull request as draft February 14, 2025 13:37
@craigcomstock craigcomstock marked this pull request as ready for review February 19, 2025 16:30
@craigcomstock

Copy link
Copy Markdown
Contributor Author

@cf-bottom jenkins please, thanks.

@cf-bottom

Copy link
Copy Markdown

@nickanderson nickanderson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me. Just a note about packaging stuff unrelated to the policy itself.

Comment thread cfe_internal/enterprise/CFE_hub_specific.cf
craigcomstock added a commit to craigcomstock/buildscripts that referenced this pull request Feb 20, 2025
Related to inhibiting management of share config.php file when a class is defined.
cfengine/masterfiles#2987

Ticket: ENT-12658
Changelog: none
@craigcomstock craigcomstock merged commit 75d54b2 into cfengine:master Feb 20, 2025
@craigcomstock craigcomstock deleted the ENT-12658/master branch February 20, 2025 22:53
@craigcomstock craigcomstock added the cherry-pick? Fixes which may need to be cherry-picked to LTS branches label Feb 20, 2025
craigcomstock added a commit to craigcomstock/buildscripts that referenced this pull request Aug 19, 2025
Related to inhibiting management of share config.php file when a class is defined.
cfengine/masterfiles#2987

Ticket: ENT-12658
Changelog: none
craigcomstock added a commit to craigcomstock/buildscripts that referenced this pull request Aug 20, 2025
… packages

To change the port, masterfiles can change these files which are distributed as part of the package so we must mark them as configuration files.
cfengine/masterfiles#2987

Ticket: ENT-12658
Changelog: none
craigcomstock added a commit to craigcomstock/buildscripts that referenced this pull request Aug 27, 2025
… packages

To change the port, masterfiles can change these files which are distributed as part of the package so we must mark them as configuration files.
cfengine/masterfiles#2987

Ticket: ENT-12658
Changelog: none
(cherry picked from commit 46ba411)
craigcomstock added a commit to craigcomstock/buildscripts that referenced this pull request Aug 27, 2025
… packages

To change the port, masterfiles can change these files which are distributed as part of the package so we must mark them as configuration files.
cfengine/masterfiles#2987

Ticket: ENT-12658
Changelog: none
(cherry picked from commit 46ba411)
@nickanderson nickanderson changed the title Inhibit management of share config.php file when mpf_disable_mission_portal_docroot_sync_from_share_gui is defined ENT-12658: Inhibit management of share config.php file when mpf_disable_mission_portal_docroot_sync_from_share_gui is defined Apr 15, 2026
@nickanderson nickanderson removed the cherry-pick? Fixes which may need to be cherry-picked to LTS branches label Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants