Skip to content

Conversation

pbartiko
Copy link
Contributor

@pbartiko pbartiko commented Aug 19, 2025

This PR creates a workflow for Betelgeuse Docstrings Validation and Automated Test Case Upload to Polarion

  • betelgeuse-dry-run: validates dosctrings (when PR is created)
  • betelgeuse-upload: generates and uploads test cases to Polarion (weekly scheduled job)

This PR is a copy of the same workflow for Insights Client: RedHatInsights/insights-client#481

Copy link
Contributor

@m-horky m-horky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have some notes, from looking at the code itself. I did not execute this against a Polarion instance.

POLARION_PASSWORD: ${{ secrets.POLARION_PASSWORD }}
run: |
curl -k -u ${{ secrets.POLARION_USERNAME }}:${{ secrets.POLARION_PASSWORD }} \
-X POST -F file=@test_case.xml \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-X POST is not necessary, it is inferred from the -F: https://daniel.haxx.se/blog/2015/09/11/unnecessary-use-of-curl-x/

run: |
PYTHONPATH=integration-tests/ betelgeuse --config-module \
custom_betelgeuse_config test-case --dry-run \
integration-tests/ dryrun_project ./test_case.xml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this is a dry run, would it make sense to expose the generated ./test_case.xml as an artifact anyone could download and inspect?

@m-horky m-horky requested a review from zpetrace August 25, 2025 12:27
@pbartiko pbartiko force-pushed the pbartiko/betelgeuse_github_workflow branch from 1182880 to 1762b7d Compare September 1, 2025 10:33
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