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

ROX-23497 Fixes version comparison for support packages #1625

Merged
merged 1 commit into from
Apr 4, 2024

Conversation

Stringy
Copy link
Collaborator

@Stringy Stringy commented Apr 4, 2024

Description

The version checking was broken which led to very small support packages for module versions that should include both up and downstream kernels. This PR fixes that check.

@Stringy Stringy requested a review from a team as a code owner April 4, 2024 14:53
@robbycochran robbycochran changed the title Fixes version comparison for support packages ROX-23497 Fixes version comparison for support packages Apr 4, 2024
@Stringy
Copy link
Collaborator Author

Stringy commented Apr 4, 2024

Verified support packages as expected - 2.7.0 with upstream and downstream drivers, 2.9.0 with only downstream:

 [16:24:15] (ghutton-mac) tmp gsutil ls -alh gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.7.0/                                                                                                                         16:24:15
      55 B  2024-04-04T15:13:17Z  gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.7.0/latest#1712243597490215  metageneration=1
     104 B  2024-04-04T15:13:17Z  gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.7.0/metadata.json#1712243597499233  metageneration=1
  1.52 GiB  2024-04-04T15:17:00Z  gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.7.0/support-pkg-2.7.0-20240404151153.zip#1712243820673919  metageneration=1
     109 B  2024-04-04T15:13:17Z  gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.7.0/support-pkg-2.7.0-20240404151153.zip.sha256#1712243597496144  metageneration=1
  1.52 GiB  2024-04-04T15:17:00Z  gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.7.0/support-pkg-2.7.0-latest.zip#1712243820468485  metageneration=1
     106 B  2024-04-04T15:13:17Z  gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.7.0/support-pkg-2.7.0-latest.zip.sha256#1712243597490376  metageneration=1
TOTAL: 6 objects, 3256103094 bytes (3.03 GiB)
➜ [16:24:26] (ghutton-mac) tmp gsutil ls -alh gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.9.0/                                                                                                                         16:24:26
      55 B  2024-04-04T15:13:17Z  gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.9.0/latest#1712243597490284  metageneration=1
     103 B  2024-04-04T15:13:17Z  gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.9.0/metadata.json#1712243597526800  metageneration=1
 37.44 MiB  2024-04-04T15:13:25Z  gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.9.0/support-pkg-2.9.0-20240404151308.zip#1712243605428657  metageneration=1
     108 B  2024-04-04T15:13:17Z  gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.9.0/support-pkg-2.9.0-20240404151308.zip.sha256#1712243597530198  metageneration=1
 37.44 MiB  2024-04-04T15:13:25Z  gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.9.0/support-pkg-2.9.0-latest.zip#1712243605433423  metageneration=1
     103 B  2024-04-04T15:13:17Z  gs://sr-roxc/collector/support-packages/giles/fix-support-package-upstream/8556760208/x86_64/2.9.0/support-pkg-2.9.0-latest.zip.sha256#1712243597522580  metageneration=1
TOTAL: 6 objects, 78507615 bytes (74.87 MiB)

@Stringy Stringy merged commit 641a536 into master Apr 4, 2024
50 of 61 checks passed
@Stringy Stringy deleted the giles/fix-support-package-upstream branch April 4, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants