Skip to content

Test octane CICD - do not merge - #9059

Open
nolanpro wants to merge 2 commits into
developfrom
test-octane-cicd
Open

nolanpro wants to merge 2 commits into
developfrom
test-octane-cicd

Conversation

@nolanpro

@nolanpro nolanpro commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

ci:deploy
ci:multitenancy
ci:k8s-branch:enable-octane-cicd

@cursor

cursor Bot commented Sep 14, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Octane worker warm/flush behavior changes request-scoped identity resolution in multitenancy, and the deploy workflow ref switch would route all PM4 builds through an experimental shared workflow if merged.

Overview
Pins the PM4 deploy workflow to the reusable workflow ref enable-octane-cicd instead of main, so CI/CD runs against the Octane-enabled pipeline (aligned with the “do not merge” test intent).

Updates config/octane.php so AnonymousUser is not warmed at worker boot when MULTITENANCY is enabled—avoiding a default-database lookup before tenant resolution—and instead adds AnonymousUser to the per-request flush list in multitenancy mode so it is resolved in the correct tenant context.

Reviewed by Cursor Bugbot for commit 0c649e9. Bugbot is set up for automated code reviews on this repo. Configure here.

@nolanpro

Copy link
Copy Markdown
Contributor Author

QA server K8S was successfully deployed https://ci-dbbc02bb39.engk8s.processmaker.net

1 similar comment
@nolanpro

Copy link
Copy Markdown
Contributor Author

QA server K8S was successfully deployed https://ci-dbbc02bb39.engk8s.processmaker.net

@nolanpro nolanpro closed this Sep 15, 2026
@nolanpro nolanpro reopened this Sep 15, 2026
@nolanpro

Copy link
Copy Markdown
Contributor Author

QA server K8S was successfully deployed https://tenant-1.ci-dbbc02bb39.engk8s.processmaker.net

@decisions-sonarqube

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://tenant-1.ci-dbbc02bb39.engk8s.processmaker.net

1 similar comment
@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://tenant-1.ci-dbbc02bb39.engk8s.processmaker.net

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.

2 participants