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

feat: use new package builder #24945

Merged
merged 2 commits into from
Apr 29, 2024
Merged

Conversation

bnpfeife
Copy link
Contributor

This uses the new ci-packager container that doesn't mangle the version strings. Some modifications to config.yaml were required, so I pushed the image to ci-packager-next. When I make all other workflows using this image compatible, I'll move everything over to ci-packager proper.

The only thing of note is that RPMs don't accept - in either the Version or Release section. So I've split the version by the first encountered -. For instance, here's what a snapshot package results with:

$ rpm -qip influxdb3-3.0.0+snapshot-fdee5321.x86_64.rpm
Name        : influxdb3
Version     : 3.0.0+snapshot
Release     : fdee5321
Architecture: x86_64
Install Date: (not installed)
Group       : Unspecified
Size        : 86751347
License     : MIT/Apache-2.0
Signature   : (none)
Source RPM  : influxdb3-3.0.0+snapshot-fdee5321.src.rpm
Build Date  : Fri 26 Apr 2024 05:23:39 PM EDT
Build Host  : 3d24177ee189
Packager    : support@influxdata.com
Vendor      : InfluxData
URL         : https://influxdata.com
Summary     : Monolithic time-series database.
Description :
Monolithic time-series database.

$ dpkg -I influxdb3_3.0.0+snapshot-fdee5321_amd64.deb 
 new Debian package, version 2.0.
 size 18204752 bytes: control archive=864 bytes.
     254 bytes,     9 lines      control              
     397 bytes,     6 lines      md5sums              
      12 bytes,     1 lines   *  postinst             #!/bin/bash
     434 bytes,    22 lines   *  preinst              #!/bin/bash
 Package: influxdb3
 Version: 3.0.0+snapshot-fdee5321
 Architecture: amd64
 Maintainer: support <support@influxdata.com>
 Installed-Size: 83934
 Section: default
 Priority: optional
 Homepage: https://influxdata.com
 Description: Monolithic time-series database.

@mgattozzi mgattozzi merged commit 8c67d99 into mgattozzi/distributables Apr 29, 2024
29 checks passed
@bnpfeife bnpfeife deleted the BNP_distributables branch May 1, 2024 14:57
mgattozzi pushed a commit that referenced this pull request May 3, 2024
* fix: use new packager to correct versions

* chore: delete foo
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