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

Introduce new composite action that can be reused from other workflows to print the debug info #949

Merged
merged 2 commits into from
Sep 13, 2022

Conversation

jkremser
Copy link
Member

@jkremser jkremser commented Sep 5, 2022

Fix #946

By default it runs with verbose being set to false, however user can re-run the action from the
github's web UI with the enable debug checkbox (see the img) being checked. Then more info is printed.

Screenshot 2022-09-05 at 14 26 41

example of usage from other workflows:

...
  - name: Print debug info
    if: always()
    uses: ./.github/actions/print-debug
...

It also adds terratests results to GITHUB_STEP_SUMMARY file, which is then visible on the workflow "landing page".

It prints:

  • events
  • evn vars
  • gslbs
  • dnsendpoints
  • metrics
  • logs
  • /etc/hosts

example runs:

Signed-off-by: Jirka Kremser [email protected]

…s to print the debug info

By default it runs with verbose being set to false, however user can re-run the action from the
github's web UI with the enable debug checkbox being checked. Then more info is printed.

example:
```yaml
...
  - name: Print debug info
    if: always()
    uses: ./.github/actions/print-debug
...
```

Signed-off-by: Jirka Kremser <[email protected]>
kuritka
kuritka previously approved these changes Sep 5, 2022
Copy link
Collaborator

@kuritka kuritka left a comment

Choose a reason for hiding this comment

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

good idea 👍

@jkremser jkremser requested a review from kuritka September 6, 2022 13:55
@jkremser jkremser added the CI all the things related to gh actions or workflow label Sep 12, 2022
@jkremser
Copy link
Member Author

@kuritka you've already approved this, but the CI was failing, because the composite action wanted to show the parsed terratest logs, but they were not parsed when called from the context of .github/workflows/terratest-more-clusters.yaml. That's why I had to add this commit 9126b18 It's a separate commit so that you can verify what has been changed since the last review and it will be squashed in the end.

@jkremser jkremser merged commit 8584eb0 into k8gb-io:master Sep 13, 2022
@jkremser jkremser deleted the debug-action branch September 13, 2022 14:33
@jkremser
Copy link
Member Author

thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI all the things related to gh actions or workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide k8gb controller metrics on GHA
2 participants