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

Release 2.2.1 #237

Merged
merged 10 commits into from
Feb 6, 2025
Merged

Release 2.2.1 #237

merged 10 commits into from
Feb 6, 2025

Conversation

m-iwanicki
Copy link
Contributor

No description provided.

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
Add help to generate-changelog script and remove container after use
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
@m-iwanicki
Copy link
Contributor Author

One test failed during Suite Setup (menus . Enter Setup Menu Tianocore And Return Construction). Not an DTS fault but I'm rerunning all tests

@DaniilKl
Copy link
Contributor

DaniilKl commented Feb 6, 2025

@m-iwanicki, what you think about adding ODROID H4 tests to CI?

@m-iwanicki
Copy link
Contributor Author

@DaniilKl good idea, didn't notice that it's not here. But I'm not sure how DPP_PASSWORD was generated so it's the same for each platform

@DaniilKl
Copy link
Contributor

DaniilKl commented Feb 6, 2025

@PLangowski, CC ^.

Current workflow design doesn't allow for running multiple workflows.
Even though runner allows only 1 job to run we can get situation in
which workflow A builds DTS and then next job is cleanup of workflow B
which will delete everything including workflow A artifacts.

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
@PLangowski
Copy link
Collaborator

@m-iwanicki I modified dts-keygen locally so that it would use a constant password

@m-iwanicki
Copy link
Contributor Author

@PLangowski Amazing. I guess that's another point to add in workflow rework issue. @PLangowski can you generate passes for ODROID?

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
@m-iwanicki
Copy link
Contributor Author

image

Fix (bug in tests): Dasharo/open-source-firmware-validation#691, tested locally

@m-iwanicki m-iwanicki merged commit 16d4fa0 into main Feb 6, 2025
12 of 13 checks 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.

3 participants