Do not copy absent logs in executeInContainer()

CentOS-PaaS-SIG / ci-pipeline

There are many projects that are concentrated on CI for Atomic Openshift. This is focused on complete CI testing of all Atomic and Base Runtime components for a new Atomic deliverable that comes from Fedora sources.

GNU General Public License v3.0

23 stars 35 forks source link

Do not copy absent logs in executeInContainer() #756

Closed Andrei-Stepanov closed 5 years ago

Andrei-Stepanov commented 5 years ago

Signed-off-by: Andrei Stepanov astepano@redhat.com

johnbieren commented 5 years ago

@joejstuart This change seems fine, right? @Andrei-Stepanov it should be fine to merge despite the stage job results. A lot of pipelines use this function so I would like Joe to agree, but seems totally sane to me

Andrei-Stepanov commented 5 years ago

@joejstuart hi!

yes, it just to remove message that is very confusing from builds logs:

15:37:20 + mv -vf logs brew-build/logs
15:37:20 mv: cannot stat 'logs': No such file or directory

johnbieren commented 5 years ago

[merge]