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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Older version returned instead of the latest one 馃槖 #17

Closed
doccaz opened this issue Sep 22, 2021 · 2 comments
Closed

Older version returned instead of the latest one 馃槖 #17

doccaz opened this issue Sep 22, 2021 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@doccaz
Copy link
Owner

doccaz commented Sep 22, 2021

Analyzing scc_ccd03b001421_210922_1732, vercheck shows an older version of dbus1 than they have installed. Also looks like its displaying the shipping version instead of updates channel:

dbus-1,1.12.2-8.6.1,1.12.2-8.3.1,Basesystem Module 15 SP1 x86_64

I assume the versions reported by repositories that match the "base_regex" have more priority over the other repositories. However, it seems I forgot to check if the version is actually higher than the original version and discard it if otherwise.

@doccaz doccaz self-assigned this Sep 23, 2021
@doccaz doccaz added the bug Something isn't working label Sep 23, 2021
@doccaz
Copy link
Owner Author

doccaz commented Sep 23, 2021

I believe I got this now. I changed the regex from:

base_regex = r"Basesystem.*"
to:
base_regex = r"(SUSE Linux Enterprise.*|Basesystem.*)"

Since the regexes in this case are evaluated in order, it should prioritize updates found in the repos beginning with "SUSE Linux Enterprise", which means the LTSS ones, before deciding on a "Basesystem" repo (if available).

I also implemented a simple version comparison function, using the LooseVersion class. So, if the results contain:

a) a package that is either in the LTSS or Basesystem repos and
b) is also higher version than the package installed on the system

then we consider the result valid as an upgrade.

@doccaz
Copy link
Owner Author

doccaz commented Sep 23, 2021

With this change, the dbus-1 package is now considered "up-to-date", as it matches the version 1.12-8.6.1, which is part of the 15 SP1 LTSS repository.

Also note that the "different versions" report shows that other LTSS updates were correctly picked up:
timezone,2021a-3.44.1,2021a-3.47.1,SUSE Linux Enterprise Server LTSS 15 SP1 x86_64

An interesting side-effect can be seen in the "different versions" report:

opensm,5.5.0.MLNX20190923.1c78385-0.1.47100,3.3.21-6.29,Server Applications Module 15 SP1 x86_64
opensm-devel,5.5.0.MLNX20190923.1c78385-0.1.47100,3.3.21-6.29,Server Applications Module 15 SP1 x86_64

which are also referenced in the "unsupported" report:

opensm,(none),5.5.0.MLNX20190923.1c78385-0.1.47100
opensm-devel,(none),5.5.0.MLNX20190923.1c78385-0.1.47100

In this case, these two packages are from an unsupported vendor. BUT, there are official versions are present in our repositories, and the supported versions are reported in the "different versions" report. This was not intended, but should be useful information to have.

@doccaz doccaz closed this as completed Sep 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant