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

Attempt nr.1 to fix #277 #279

Closed

Conversation

AlexRuiz7
Copy link
Member

@AlexRuiz7 AlexRuiz7 commented Jun 24, 2024

Description

On this PR I try to fix the missing revision on the DEB package metadata by removing the -ui option when invoking debuild, to make it identical to the previous package builder.

https://man7.org/linux/man-pages/man1/dpkg-buildpackage.1.html

Issues Resolved

#277

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)
  • Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@AlexRuiz7 AlexRuiz7 requested a review from a team as a code owner June 24, 2024 14:02
@f-galland
Copy link
Member

I tested this but I still cannot see the revision in the final package:

fede@tyner:~/src/wazuh-indexer (ci/277-add-revision-as-metadata-to-deb-packages)
$ dpkg -I artifacts/dist/wazuh-indexer_4.9.0_amd64.deb 
 new Debian package, version 2.0.
 size 850530026 bytes: control archive=27753 bytes.
    1704 bytes,    30 lines      conffiles            
     676 bytes,    13 lines      control              
  104140 bytes,  1003 lines      md5sums              
    1602 bytes,    58 lines   *  postinst             #!/bin/bash
    1118 bytes,    31 lines   *  preinst              #!/bin/bash
     828 bytes,    26 lines   *  prerm                #!/bin/bash
 Package: wazuh-indexer
 Version: 4.9.0
 Architecture: amd64
 Maintainer: Wazuh Indexer Team <info@wazuh.com>
 Installed-Size: 1051646
 Section: web
 Priority: optional
 Homepage: https://www.wazuh.com/
 Description: Wazuh indexer is a near real-time full-text search and analytics engine that gathers security-related data into one platform.
  This Wazuh central component indexes and stores alerts generated by the Wazuh server.
  Wazuh indexer can be configured as a single-node or multi-node cluster, providing scalability and high availability.
  Documentation can be found at https://documentation.wazuh.com/current/getting-started/components/wazuh-indexer.html
 License: Apache-2.0

@f-galland

This comment was marked as duplicate.

@AlexRuiz7
Copy link
Member Author

Didn't work

dpkg-deb -I wazuh-indexer_4.9.0-0_amd64_9fe851b.deb 
 paquete Debian nuevo, versión 2.0.
 tamaño 850267444 bytes: archivo de control= 27750 bytes.
    1704 bytes,    30 líneas     conffiles            
     676 bytes,    13 líneas     control              
  104140 bytes,  1003 líneas     md5sums              
    1602 bytes,    58 líneas  *  postinst             #!/bin/bash
    1118 bytes,    31 líneas  *  preinst              #!/bin/bash
     828 bytes,    26 líneas  *  prerm                #!/bin/bash
 Package: wazuh-indexer
 Version: 4.9.0
 Architecture: amd64
 Maintainer: Wazuh Indexer Team <info@wazuh.com>
 Installed-Size: 1051397
 Section: web
 Priority: optional
 Homepage: https://www.wazuh.com/
 Description: Wazuh indexer is a near real-time full-text search and analytics engine that gathers security-related data into one platform.
  This Wazuh central component indexes and stores alerts generated by the Wazuh server.
  Wazuh indexer can be configured as a single-node or multi-node cluster, providing scalability and high availability.
  Documentation can be found at https://documentation.wazuh.com/current/getting-started/components/wazuh-indexer.html
 License: Apache-2.0

@AlexRuiz7 AlexRuiz7 closed this Jun 24, 2024
@AlexRuiz7 AlexRuiz7 deleted the ci/277-add-revision-as-metadata-to-deb-packages branch June 24, 2024 16:20
@AlexRuiz7 AlexRuiz7 self-assigned this Jun 24, 2024
@AlexRuiz7 AlexRuiz7 changed the title Test to fix #277 Attempt nr.1 to fix #277 Jun 24, 2024
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.

The Wazuh indexer package does not set revision in the package metadata
2 participants