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

patch: Add Allure Report beta #131

Merged
merged 15 commits into from
Jan 26, 2024
Merged

patch: Add Allure Report beta #131

merged 15 commits into from
Jan 26, 2024

Conversation

carlcsaposs-canonical
Copy link
Contributor

@carlcsaposs-canonical carlcsaposs-canonical commented Jan 25, 2024

Usage

Warning

This feature is in beta and not part of the public interface. It is subject to breaking changes or removal on a patch version bump.

  1. poetry add --group integration allure-pytest
  2. Set _beta_allure_report: true for one instance of integration_test_charm.yaml. If integration_test_charm.yaml is called with a matrix, _beta_allure_report can only be true for one combination of the matrix.
  3. Add permission to integration_test_charm.yaml job and all calling workflows
    permissions:
      contents: write  # Needed for Allure Report beta
  1. Create gh pages branch
    # For first run, manually create branch with no history
    # (e.g.
    # git checkout --orphan gh-pages-beta
    # git rm -rf .
    # touch .nojekyll
    # git add .nojekyll
    # git commit -m "Initial commit"
    # git push origin gh-pages-beta
    # )
  2. Enable gh pages publishing at https://github.com/canonical/mysql-router-k8s-operator/settings/pages (replace mysql-router-k8s-operator with repository name)
    gh-pages

Example for 1-3: canonical/mysql-router-k8s-operator#198

@carlcsaposs-canonical
Copy link
Contributor Author

tested on canonical/mysql-router-k8s-operator#196

@carlcsaposs-canonical carlcsaposs-canonical changed the title patch: Add Allure report beta patch: Add Allure Report beta Jan 25, 2024
@carlcsaposs-canonical carlcsaposs-canonical force-pushed the allure-report branch 2 times, most recently from 1e04b7f to faf724f Compare January 25, 2024 14:02
@carlcsaposs-canonical carlcsaposs-canonical marked this pull request as ready for review January 25, 2024 14:06
@paulomach
Copy link

paulomach commented Jan 25, 2024

How to access the allure result page? Found it

Copy link

@paulomach paulomach left a comment

Choose a reason for hiding this comment

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

Nice. Next time test on the server. Router is too boring with all green

Copy link

@taurus-forever taurus-forever left a comment

Choose a reason for hiding this comment

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

Exceptionally great! I like it! Thank you!

@carlcsaposs-canonical carlcsaposs-canonical merged commit 1b00758 into main Jan 26, 2024
1 check passed
@carlcsaposs-canonical carlcsaposs-canonical deleted the allure-report branch January 26, 2024 09:09
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.

3 participants