Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

[1LP][RFR] Fixed vm retire now error message in logs #9831

Merged
merged 1 commit into from
Jan 17, 2020

Conversation

ganeshhubale
Copy link
Member

Signed-off-by: Ganesh Hubale [email protected]

Purpose or Intent

  • Added specific error in failures patterns
  • Fixed error:
>               raise FailPatternMatchError(pattern, "Expected failure pattern found in log.", line)
E               cfme.utils.log_validator.FailPatternMatchError: "Pattern '.*ERROR.*': Expected failure pattern found in log."

cfme/utils/log_validator.py:82: FailPatternMatchError
FailPatternMatchError
b'"Pattern \'.*ERROR.*\': Expected failure pattern found in log."'

PRT Run

{{pytest: cfme/tests/automate/test_service_automate.py -k 'test_retire_vm_now' --use-template-cache -qsvv --long-running --use-provider=complete }}

@ganeshhubale ganeshhubale changed the title [WIPTEST] Fixed vm retire now error message in logs [RFR] Fixed vm retire now error message in logs Jan 10, 2020
@sshveta sshveta changed the title [RFR] Fixed vm retire now error message in logs [1LP][RFR] Fixed vm retire now error message in logs Jan 13, 2020
@dajoRH dajoRH changed the title [1LP][RFR] Fixed vm retire now error message in logs [1LP][WIP] Fixed vm retire now error message in logs Jan 14, 2020
@ganeshhubale ganeshhubale changed the title [1LP][WIP] Fixed vm retire now error message in logs [1LP][WIPTEST] Fixed vm retire now error message in logs Jan 17, 2020
@ganeshhubale ganeshhubale changed the title [1LP][WIPTEST] Fixed vm retire now error message in logs [1LP][RFR] Fixed vm retire now error message in logs Jan 17, 2020
Copy link
Contributor

@digitronik digitronik left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@digitronik digitronik self-assigned this Jan 17, 2020
@digitronik digitronik merged commit ba4e02e into ManageIQ:master Jan 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants