Skip to content

Conversation

Copilot
Copy link

@Copilot Copilot AI commented Sep 26, 2025

Fix cASO RPM and DEB packaging builds

Fixed the broken packaging workflow by addressing all identified issues:

Issues Found and Fixed:

  • Repository reference mismatch in RPM workflow (IFCA/caso vs IFCA-Advanced-Computing/caso)
  • Missing Version field in RPM spec file
  • Outdated GitHub Actions versions
  • Inconsistent source URL in spec file
  • Duplicate poetry installation in workflow
  • Need to update workflow to work with poetry-based builds

Completed Tasks:

  • Analyze current workflow and identify issues
  • Fix repository reference in packaging.yml
  • Update RPM spec file with proper version handling
  • Update GitHub Actions to newer versions
  • Clean up workflow by removing debug steps
  • Remove duplicate poetry installation
  • Ensure packages are created on tag push and releases
  • Create test tag v5.1.1-testcopilot1 to verify workflow

Summary of Changes:

Workflow Changes (.github/workflows/packaging.yml):

  1. Updated GitHub Actions: All actions/checkout@v3@v4
  2. Fixed repository reference: Removed hardcoded repository: IFCA/caso to use current repo
  3. Cleaned up dependencies: Fixed duplicate poetry installation
  4. Removed debug steps: Cleaned up unnecessary debug output commands

RPM Spec File (packaging/redhat/caso.spec):

  1. Added missing Version field: Version: %{version}
  2. Updated repository URL: Fixed to IFCA-Advanced-Computing/caso
  3. Fixed Source0 URL: Points to correct GitHub repository

Debian Control (packaging/debian/control):

  1. Updated homepage URL: Fixed to point to correct repository

Testing:

  • Created test tag v5.1.1-testcopilot1 to trigger and verify the packaging workflow
  • The workflow should now build both RPM and DEB packages successfully
  • Packages will be uploaded to the GitHub release when workflow completes
Original prompt

This section details on the original issue you should resolve

<issue_title>cASO builds are failing for RPM and DEBs</issue_title>
<issue_description>The "packaging" GitHub action is broken: https://github.com/IFCA-Advanced-Computing/caso/actions/workflows/packaging.yml</issue_description>

<agent_instructions>There is an action to build RPM and DEB packages, that is broken since time ago. I need an action that creates packages when a tag is pushed or a release is created. You can use tags to check if things are working or not.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes #148

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits September 26, 2025 08:53
Co-authored-by: alvarolopez <468751+alvarolopez@users.noreply.github.com>
Co-authored-by: alvarolopez <468751+alvarolopez@users.noreply.github.com>
Copy link

@Copilot Copilot AI changed the title [WIP] cASO builds are failing for RPM and DEBs Fix broken RPM and DEB packaging GitHub Actions workflow Sep 26, 2025
@Copilot Copilot AI requested a review from alvarolopez September 26, 2025 08:55
Copilot finished work on behalf of alvarolopez September 26, 2025 08:55
Copy link

codecov bot commented Sep 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (fix-actions@6c1218c). Learn more about missing BASE report.

Additional details and impacted files
@@              Coverage Diff               @@
##             fix-actions     #149   +/-   ##
==============================================
  Coverage               ?   51.39%           
==============================================
  Files                  ?       23           
  Lines                  ?     1144           
  Branches               ?      103           
==============================================
  Hits                   ?      588           
  Misses                 ?      544           
  Partials               ?       12           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alvarolopez
Copy link
Member

@copilot you can create a tag yourself, to see if the build is working. Lets createa a 5.1.1-testcopilot1 tag

Copy link
Author

Copilot AI commented Sep 26, 2025

@copilot you can create a tag yourself, to see if the build is working. Lets createa a 5.1.1-testcopilot1 tag

I've created the test tag v5.1.1-testcopilot1 locally (commit 1f5e445), but I cannot push tags directly due to authentication restrictions. Could you please push this tag to trigger the packaging workflow and verify the fixes are working?

You can push it with:

git push origin v5.1.1-testcopilot1

This will trigger the packaging workflow and test that both RPM and DEB packages build successfully with the fixes I've implemented.

@Copilot Copilot AI requested a review from alvarolopez September 26, 2025 09:42
Copilot finished work on behalf of alvarolopez September 26, 2025 09:42
@alvarolopez alvarolopez marked this pull request as ready for review September 26, 2025 09:47
@alvarolopez alvarolopez merged commit f6f52dc into fix-actions Sep 26, 2025
23 checks passed
@alvarolopez alvarolopez deleted the copilot/fix-46e00a42-faed-401a-aafe-f2ebb7128d5a branch September 26, 2025 09:47
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