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

Enable pre-merge HW boot tests #121

Merged
merged 1 commit into from
Jan 20, 2025
Merged

Enable pre-merge HW boot tests #121

merged 1 commit into from
Jan 20, 2025

Conversation

vjuntunen
Copy link
Contributor

Enable HW boot test to pre-merge pipeline

@vjuntunen vjuntunen requested a review from a team January 16, 2025 11:41
@henrirosten
Copy link
Collaborator

@vjuntunen: Please reference test results if you have tested this.

@henrirosten
Copy link
Collaborator

On line 12, I think you want to set this to true:

def purge_stashed_artifacts = false

Otherwise, we would keep all the images ever build on all PR pushes, which would explode the storage use.

@vjuntunen
Copy link
Contributor Author

Oh yes I enabled that purge. And maybe we wait for next pre-merge run and take that as reference if succeeded.

@henrirosten
Copy link
Collaborator

@vjuntunen: I think you are testing this in prod, right?

I see your changes add an 'Artifacts' link on pre-merge builds, but the link is now broken because you purge them.

@vjuntunen
Copy link
Contributor Author

Yes in prod, because we do not have that static system to test that in dev. Should that purge delete that link also? I just use it here.. But now boot tests are all green for pre-merge line: https://ghaf-jenkins-controller-prod.northeurope.cloudapp.azure.com/job/ghaf-pre-merge-pipeline/319/

@vjuntunen
Copy link
Contributor Author

Enable HW boot test to pre-merge pipeline

Signed-off-by: Ville-Pekka Juntunen <[email protected]>
Copy link
Collaborator

@henrirosten henrirosten left a comment

Choose a reason for hiding this comment

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

LGTM.

For any upcoming changes, please find a way to test this in dev or your private deployment.

@vjuntunen vjuntunen merged commit 64bc517 into tiiuae:main Jan 20, 2025
1 check passed
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