Skip to content

[CI] Add setup.py pycti/connectors-sdk replacement for rolling and prerelease builds#6500

Draft
Copilot wants to merge 3 commits into
masterfrom
copilot/fix-rolling-image-pycti-master
Draft

[CI] Add setup.py pycti/connectors-sdk replacement for rolling and prerelease builds#6500
Copilot wants to merge 3 commits into
masterfrom
copilot/fix-rolling-image-pycti-master

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 21, 2026

Rolling and prerelease Docker builds correctly rewrite pycti==<version> to a git-ref URL in requirements.txt and pyproject.toml, but setup.py was never handled — connectors using setup.py for dependencies would silently ship the pinned PyPI release instead of the master/branch ref.

Changes

  • .circleci/templates/dynamic.yml.j2

    • Added find . -name setup.py -exec sed … replacement commands for both pycti and connectors-sdk in the prerelease (uses $CIRCLE_BRANCH) and rolling (uses $RELEASE_REF) build blocks, matching the existing pattern used for pyproject.toml
    • Removed a stray bare URL line in the prerelease block — a bash no-op leftover that caused a silent git: unknown subcommand error on every prerelease build
  • .gitignore — added dynamic.yml (the generated CI continuation config) to prevent it from being accidentally committed

Pattern applied to setup.py (rolling example)

find . -name setup.py -exec sed \
  "s|pycti==.*$|pycti @ git+https://github.com/OpenCTI-Platform/opencti.git@$RELEASE_REF#subdirectory=client-python\",|" \
  -i {} \;
find . -name setup.py -exec sed \
  "s|connectors-sdk.*$|connectors-sdk @ git+https://github.com/OpenCTI-Platform/connectors.git@$RELEASE_REF#subdirectory=connectors-sdk\",|" \
  -i {} \;

Copilot AI and others added 2 commits May 21, 2026 22:12
Copilot AI changed the title [WIP] Fix rolling image to use pycti from master branch [CI] Add setup.py pycti/connectors-sdk replacement for rolling and prerelease builds May 21, 2026
Copilot AI requested a review from SamuelHassine May 21, 2026 22:13
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.

[ALL] Rolling image should always be built with pycti master

2 participants