Skip to content

Conversation

@Magisus
Copy link
Contributor

@Magisus Magisus commented Nov 6, 2025

This commit undoes a previous change that made the run_cd4pe_job task private. This hid the task from the PE console, which was desirable so that users would not be able to run it directly, outside of CD. However, with this change, users of PE 2025.6 were also no longer able to give the CD service account user permission to run this task, meaning that all CD jobs would fail. So until we figure out a better long-term solution, this commit makes the task public again.

This commit also updates the task description to make it more clear how it is intended to be run, and to discourage users from running it directly.

This commit undoes a previous change that made the `run_cd4pe_job` task
private. This hid the task from the PE console, which was desirable so
that users would not be able to run it directly, outside of CD. However,
with this change, users of PE 2025.6 were also no longer able to
give the CD service account user permission to run this task, meaning
that all CD jobs would fail. So until we figure out a better long-term
solution, this commit makes the task public again.

This commit also updates the task description to make it more clear how
it is intended to be run, and to discourage users from running it
directly.
@Magisus Magisus requested a review from a team as a code owner November 6, 2025 22:15
@Magisus Magisus merged commit 445f778 into master Nov 7, 2025
5 checks passed
@Magisus Magisus deleted the remove-internal branch November 7, 2025 18:53
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.

3 participants