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

Smart build in buildimages #578

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

Smart build in buildimages #578

wants to merge 11 commits into from

Conversation

chouetz
Copy link
Member

@chouetz chouetz commented Apr 22, 2024

Introduced new rule:changes:path on all the build jobs of the gitlab pipeline, to build only the required images, based on modified content.

As a consequence, I also modified the trigger_test to adapt with this smart build:

  • each build pushes a file as artefact to warn it was executed
  • the generate_tests job will fill-in the newly created trigger_template.yml with only the required images
  • the trigger_datadog_agent_tests job runs the generated file as a child pipeline.

I did several tests to validate this change [WIP]:

  • I validated the child pipeline correctly triggers a datadog-agent pipeline with the created images
  • I did several "fake" changes to see the triggers:
    • on the README.md file -> no build
    • on the python requirements -> blabla
    • only the system probe

@chouetz chouetz changed the title reorder build jobs Smart build in buildimages Apr 23, 2024
@chouetz chouetz force-pushed the nschweitzer/dynamic_build branch from 9903180 to a58e4e7 Compare October 24, 2024 14:22
tasks/gatekeeper.py Outdated Show resolved Hide resolved
tasks/gatekeeper.py Outdated Show resolved Hide resolved
@chouetz chouetz force-pushed the nschweitzer/dynamic_build branch from a58e4e7 to 4351df8 Compare October 24, 2024 14:26
@chouetz chouetz force-pushed the nschweitzer/dynamic_build branch from e17c71c to 9f056f1 Compare October 24, 2024 15:49
- Fix the task name typo
- Aligned the variable names to the IMAGE name used in gitlab
  configuration
- Add tests for the parts of the generate task
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