• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

kubevirt / hyperconverged-cluster-operator / 3796
81%
main: 76%

Build:
Build:
LAST BUILD BRANCH: bump-kubevirtci
DEFAULT BRANCH: main
Ran 24 Sep 2020 08:33AM UTC
Jobs 1
Files 3
Run time 1s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

24 Sep 2020 08:27AM UTC coverage: 80.722%. Remained the same
3796

push

travis-ci

web-flow
[Release 1.2] Fix CSV merge issues (#831)

* Optionally ignore relatedImages from component operators (#829)

Adda a flag to the csv-merger tool to optionally ignore
relatedImages from component operators generated CSV.

Signed-off-by: Simone Tiraboschi <stirabos@redhat.com>

* add error handling to hack/build-manifest.sh (#828)

* add error handling to hack/build-manifest.sh

`set -e` does not stop the process if the error is happened within a command substitution. This is why the [`hack/build-manifest.sh`] script didn't fail in some cases, e.g. when an image was missing in the remote registry.

This PR added error handling to the script:
* Added `trap` command and the `catch` method
* command substitution inside string building didn't catch by the trap. This lines were split to make sure the trap catches any error. For example:
```bash
A_VARIABLE="Text text${VAR1}/$(failed command)"
```
was split to something like:
```bash
A_TEMP_VAR="$(failed command)"
A_VARIABLE="Text text${VAR1}/${A_TEMP_VAR}"
```
* Single line local variables with command substitution were not caught by the trap, and was also split. For example:
```bash
function foo() {
  ...
  local a_variable="$(failed command)"
  ...
```
was split to something like:
```bash
function foo() {
  ...
  local a_variable
  a_variable="$(failed command)"
  ...
```
* Add simple versifcation to the operator CSV files, by greping for their `image:` line. This check both that the file is not empty and the image makes some sense

Signed-off-by: Nahshon Unna-Tsameret <nunnatsa@redhat.com>

* csv-merger will now panic if one of the CSVs is empty

Signed-off-by: Nahshon Unna-Tsameret <nunnatsa@redhat.com>

Co-authored-by: Simone Tiraboschi <stirabos@redhat.com>

1319 of 1634 relevant lines covered (80.72%)

39.28 hits per line

Jobs
ID Job ID Ran Files Coverage
1 3796.1 24 Sep 2020 08:33AM UTC 0
80.72
Travis Job 3796.1
Source Files on build 3796
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #3796
  • 188cecf0 on github
  • Prev Build on release-1.2 (#3780)
  • Next Build on release-1.2 (#3901)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc