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

Remove duplicated build for Lucy in CI #14267

Merged
merged 3 commits into from
Oct 6, 2023

Conversation

dkijania
Copy link
Member

@dkijania dkijania commented Oct 4, 2023

In our CI pipeline we are building test executive twice. First time in build-artifacts.sh script which is launched per every codename (Bullseye/Focal/Buster) and publishing to our debian repo. Second, in separate job for test executive build which puts exe on s3 storage which then is taken by execute tests jobs. My change was to rely in test execute job on deb package rather than cached binary from s3

@dkijania dkijania changed the base branch from develop to berkeley October 4, 2023 21:15
@dkijania
Copy link
Member Author

dkijania commented Oct 5, 2023

!ci-build-me

@dkijania
Copy link
Member Author

dkijania commented Oct 5, 2023

!ci-nightly-me

@dkijania
Copy link
Member Author

dkijania commented Oct 5, 2023

@dkijania dkijania changed the title Dkijania/do not build te again Remove duplicated build for Lucy in CI Oct 5, 2023
@dkijania dkijania marked this pull request as ready for review October 5, 2023 09:48
@dkijania dkijania requested a review from a team as a code owner October 5, 2023 09:48
@dkijania dkijania self-assigned this Oct 5, 2023
@dkijania
Copy link
Member Author

dkijania commented Oct 5, 2023

!ci-build-me

@dkijania
Copy link
Member Author

dkijania commented Oct 6, 2023

!ci-build-me

@deepthiskumar
Copy link
Member

!approved-for-mainnet

@dkijania dkijania merged commit d731e7a into berkeley Oct 6, 2023
2 checks passed
@dkijania dkijania deleted the dkijania/do_not_build_te_again branch October 6, 2023 20: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.

3 participants