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

New branch to investigate why CI/CD failing in PR #7

Open
wants to merge 143 commits into
base: merge-check
Choose a base branch
from

Conversation

MukuFlash03
Copy link
Owner

Original PR (link)

Failed run (link)

Mahadik, Mukul Chandrakant and others added 25 commits July 16, 2024 11:48
The previous PR commit history got messed up due to rebasing to handle the commits that had missing signoffs.
Created a new PR to bring over the final changes from there.

PR link: EVerest#62

Signed-off-by: Mahadik, Mukul Chandrakant <[email protected]>
…x job

Found out an issue with the latest tagged image not being fetched in the docker compose command.
The reason was that the matrix job only loaded its respective image from the cache.
But the docker compose command for running automated tests uses both manager and mqtt server images.

To solve this, initially I used docker build push action to reload the mqtt image in the matrix job for manager.
But then finally went with the approach of using artifacts to share the mqtt image between jobs - uploaded in mqtt matrix job, downloaded in manager matrix job.

Signed-off-by: Mahadik, Mukul Chandrakant <[email protected]>
Signed-off-by: Mahadik, Mukul Chandrakant <[email protected]>
Signed-off-by: Mukul Chandrakant Mahadik <[email protected]>
EVEREST_VERSION=2024.7.1 ran almost close till the end.
But, got an insufficient space; no space left; error.

Run:
https://github.com/MukuFlash03/everest-demo/actions/runs/10358908297/job/28674141266
This reverts commit 0b1273c.
Added the hardcoded build-kit alpine image with the EVEREST_VERSION = 2024.3.0 and matching SHA256 digest to the previous latest tag version which has now been moved to untagged (7494bd6624aee3f882b4f1edbc589879e1d6d0ccc2c58f3f5c87ac1838ccd1de)

Found this image by going through the logs in the last successful run (in my forked repo) and looking at the FROM layer in the Docker build step.
fa60246

Image link:
https://github.com/everest/everest-ci/pkgs/container/build-kit-alpine/161694439

---

Expected: Should pass since it matches configurations of last successful run with matching EVEREST_VERSION and SHA256 digest for build-kit alpine base image.

----

Will add more details in PR comments.

----

Signed-off-by: Mahadik, Mukul Chandrakant <[email protected]>
@MukuFlash03 MukuFlash03 force-pushed the run-automated-tests-check branch from e4268a9 to f364e0d Compare September 24, 2024 08:18
Mahadik, Mukul Chandrakant added 30 commits September 25, 2024 13:03
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