Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI] Add container build #804

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

Renizmy
Copy link

@Renizmy Renizmy commented Dec 21, 2024

Create a Connector Base Image (WIP 🚧)

  • Should follow the same lifecycle as the Python library.
  • ARM support has been implemented.

Required environment variables for docker registry connection

  • DOCKERHUB_PASS
  • DOCKERHUB_USERNAME

Completed Tasks

  • Python 3.12 build
  • Python 3.11 build
  • ARM image support
  • Use a dedicated app user instead of the default image user

Pending Tasks

  • Sign images
  • Add solution to manually trigger nightly builds
  • Implement vulnerability scanner (with cache optimization)

@helene-nguyen helene-nguyen self-assigned this Jan 15, 2025
Copy link
Member

@helene-nguyen helene-nguyen left a comment

Choose a reason for hiding this comment

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

@Renizmy Thanks a lot for your contribution !

I have added some questions and changes to do.

And to move forward, we also need to create an image with rolling tag, could you add these changes too on the file ?

name: Build opencti/python-client
command: |
docker run --privileged --rm tonistiigi/binfmt --install all
CIRCLE_TAG=${CIRCLE_TAG:-nightly}
Copy link
Member

Choose a reason for hiding this comment

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

@Renizmy Thank you for this PR, could you explain this ? What is the purpose of CIRCLE_TAG:-nightly ?

--build-arg BASE_IMAGE="python:3.12-alpine3.20"

- run:
name: Push image to regsitry
Copy link
Member

Choose a reason for hiding this comment

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

Could you correct the naming ?

Suggested change
name: Push image to regsitry
name: Push image to registry

--build-arg BASE_IMAGE="python:3.11-alpine3.20"

- run:
name: Push image to regsitry
Copy link
Member

Choose a reason for hiding this comment

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

Could you correct the naming ?

Suggested change
name: Push image to regsitry
name: Push image to registry

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