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:skip-build

@cursor

cursor Bot commented Sep 14, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Octane worker warm/flush changes alter how tenant-scoped AnonymousUser is cached across requests; incorrect behavior could affect anonymous auth under multitenancy. The workflow ref pin is experimental and should not ship to production branches.

Overview
CI: The deploy-pm4 reusable workflow is temporarily pointed at processmaker/.github@enable-octane-cicd instead of @main so Octane-related deploy/CI behavior can be exercised on this branch (marked do-not-merge).

Octane: When MULTITENANCY is enabled, AnonymousUser is no longer pre-warmed at worker boot and is added to per-request flush bindings. Warming AnonymousUser at boot was hitting the default DB connection before tenant resolution; flushing it under multitenancy forces a fresh resolution per request.

Reviewed by Cursor Bugbot for commit a7a38da. 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
@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

@nolanpro

Copy link
Copy Markdown
Contributor Author

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.

1 participant