Skip to content

Ignore some errors in order to allow successful execution#13

Merged
nmorey merged 2 commits intoSUSE:masterfrom
frankenmichl:less_fails
Jan 8, 2021
Merged

Ignore some errors in order to allow successful execution#13
nmorey merged 2 commits intoSUSE:masterfrom
frankenmichl:less_fails

Conversation

@frankenmichl
Copy link
Copy Markdown
Collaborator

Two minor fixes allowing the tests to be executed properly:

  1. Don't fail the run if we cannot reset all ports. We can ignore this.
  2. Ignore an error from sed and allow the SRP test to execute

If we reset the ports, sometimes it does not work as it should. However,
we should not fail the entire run when we cannot reset the ports and do
proper cleanup here.

Signed-off-by: Michael Moese <mmoese@suse.com>
@frankenmichl frankenmichl requested a review from nmorey January 8, 2021 06:44
In case we replace $server_sysguid with $server_guid, we sometimes get
an EEXISTS. However, we should not bother, as the result in this case is
the same - so we can just continue here.

Signed-off-by: Michael Moese <mmoese@suse.com>
@nmorey nmorey merged commit 980d102 into SUSE:master Jan 8, 2021
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.

2 participants