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

fix(ci): the latest package version coincides with the version apt would choose #12909

Merged
merged 1 commit into from Jun 3, 2022

Conversation

nstng
Copy link
Contributor

@nstng nstng commented Jun 3, 2022

Signed-off-by: Nils Semmelrock nils.semmelrock@tngtech.com

Summary

In pydep at one point for a package python3-<NAME> it is checked, what is the latest version that could be installed via apt. The latest version is calculated manually via "<"-comparison. This fails e.g., for python3-webcolors when it is available in versions 1.5-2.1 and 1.11.1. The algorithm returns 1.5-2.1 while apt would install 1.11.1 as the latest version.

Here: do not calculate the latest version manually, but use the existing apt.package.candidate.version mechanics (candidate ^= the candidate of a package that would be installed by apt).

Test Plan

CI - build-all/agw-build/Build AGW
But to be honest, this is hard to reproduce because this relies on certain repository states.

Additional Information

  • This change is backwards-breaking

…uld choose

Signed-off-by: Nils Semmelrock <nils.semmelrock@tngtech.com>
@nstng nstng requested a review from a team as a code owner June 3, 2022 07:47
@nstng nstng requested a review from sebathomas June 3, 2022 07:47
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines. label Jun 3, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jun 3, 2022

Thanks for opening a PR! 💯

A couple initial guidelines

Howto

  • Reviews. The "Reviewers" listed for this PR are the Magma maintainers who will shepherd it.
  • Checks. All required CI checks must pass before merge.
  • Merge. Once approved and passing CI checks, use the ready2merge label to indicate the maintainers can merge your PR.

More info

Please take a moment to read through the Magma project's

If this is your first Magma PR, also consider reading

@github-actions github-actions bot added the component: ci All updates on CI (Jenkins/CircleCi/Github Action) label Jun 3, 2022
@nstng nstng linked an issue Jun 3, 2022 that may be closed by this pull request
@nstng nstng requested a review from tmdzk June 3, 2022 07:50
@nstng
Copy link
Contributor Author

nstng commented Jun 3, 2022

@tmdzk FYI one issue with the build_all workflow.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 3, 2022

feg-workflow

    2 files  202 suites   37s ⏱️
371 tests 371 ✔️ 0 💤 0
385 runs  385 ✔️ 0 💤 0

Results for commit f543224.

@tmdzk tmdzk merged commit 334c4a3 into magma:master Jun 3, 2022
emakeev pushed a commit to emakeev/magma that referenced this pull request Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: ci All updates on CI (Jenkins/CircleCi/Github Action) size/S Denotes a PR that changes 10-29 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Analyse and Fix build-all/agw-build/Build AGW
2 participants