Skip to content

fix(ci): use multiline syntax to print logs on failure#2637

Merged
simar7 merged 1 commit intoaquasecurity:mainfrom
nikpivkin:gh-act-multiline
Jul 3, 2025
Merged

fix(ci): use multiline syntax to print logs on failure#2637
simar7 merged 1 commit intoaquasecurity:mainfrom
nikpivkin:gh-act-multiline

Conversation

@nikpivkin
Copy link
Copy Markdown
Contributor

Description

Fixes an error where multiple commands were joined into one, causing kubectl to misinterpret flags.

Run kubectl logs -n trivy-system deployment/trivy-operator echo "reports:" kubectl get clustervulnerabilityreports.aquasecurity.github.io -A
  kubectl logs -n trivy-system deployment/trivy-operator echo "reports:" kubectl get clustervulnerabilityreports.aquasecurity.github.io -A
  shell: /usr/bin/bash -e {0}
  env:
    KIND_VERSION: v0.[2](https://github.com/aquasecurity/trivy-operator/actions/runs/16005323687/job/45151546248?pr=2631#step:20:2)9.0
    KIND_IMAGE: kindest/node:v1.33.1@sha256:050072256b9a903bd914c0b2866828150cb229cea0efe5892e2b644d5dd3b34f
    DOCKER_CLI_EXPERIMENTAL: enabled
error: unknown shorthand flag: 'A' in -A
See 'kubectl logs --help' for usage.

Checklist

  • I've read the guidelines for contributing to this repository.
  • I've added tests that prove my fix is effective or that my feature works.
  • I've updated the documentation with the relevant information (if needed).
  • I've added usage information (if the PR introduces new options)
  • I've included a "before" and "after" example to the description (if the PR is a user interface change).

@simar7 simar7 merged commit f55b1fc into aquasecurity:main Jul 3, 2025
8 checks passed
@simar7
Copy link
Copy Markdown
Member

simar7 commented Jul 3, 2025

I think this PR has exposed a bug: https://github.com/aquasecurity/trivy-operator/actions/runs/16034131780/job/45243128262?pr=2638

Run kubectl logs -n trivy-system deployment/trivy-operator echo "reports:" kubectl get clustervulnerabilityreports.aquasecurity.github.io -A
error: unknown shorthand flag: 'A' in -A
See 'kubectl logs --help' for usage.
Error: Process completed with exit code 1.

@nikpivkin nikpivkin deleted the gh-act-multiline branch July 3, 2025 05:46
@nikpivkin
Copy link
Copy Markdown
Contributor Author

I think this PR has exposed a bug: https://github.com/aquasecurity/trivy-operator/actions/runs/16034131780/job/45243128262?pr=2638

Run kubectl logs -n trivy-system deployment/trivy-operator echo "reports:" kubectl get clustervulnerabilityreports.aquasecurity.github.io -A
error: unknown shorthand flag: 'A' in -A
See 'kubectl logs --help' for usage.
Error: Process completed with exit code 1.

Doesn't this PR fix that?

simar7 pushed a commit that referenced this pull request Jul 8, 2025
simar7 added a commit that referenced this pull request Jul 8, 2025
* release: prepare v0.27.2

* build(deps): bump github.com/go-viper/mapstructure/v2 (#2630)

Bumps [github.com/go-viper/mapstructure/v2](https://github.com/go-viper/mapstructure) from 2.2.1 to 2.3.0.
- [Release notes](https://github.com/go-viper/mapstructure/releases)
- [Changelog](https://github.com/go-viper/mapstructure/blob/main/CHANGELOG.md)
- [Commits](go-viper/mapstructure@v2.2.1...v2.3.0)

---
updated-dependencies:
- dependency-name: github.com/go-viper/mapstructure/v2
  dependency-version: 2.3.0
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix(ci): use multiline syntax to print logs on failure (#2637)

Signed-off-by: Nikita Pivkin <[email protected]>

* docs: fix typo in RELEASING.md for Helm chart file name (#2642)

Signed-off-by: Nikita Pivkin <[email protected]>

* chore: bump up UBI version to 9.6-1751445649 (#2643)

* ci: add ubi images for vuln pre-release scanning (#2628)

* fix: correct prev_git_tag

---------

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Nikita Pivkin <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Nikita Pivkin <[email protected]>
Co-authored-by: afdesk <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants