Skip to content

test: Use grep -E variant to be more generic and update some patterns#23991

Closed
dzafman wants to merge 1 commit intoceph:masterfrom
dzafman:wip-35845
Closed

test: Use grep -E variant to be more generic and update some patterns#23991
dzafman wants to merge 1 commit intoceph:masterfrom
dzafman:wip-35845

Conversation

@dzafman
Copy link
Contributor

@dzafman dzafman commented Sep 7, 2018

Fixes: http://tracker.ceph.com/issues/35845

Signed-off-by: David Zafman dzafman@redhat.com

  • References tracker ticket

@dzafman dzafman added the tests label Sep 7, 2018
@dzafman dzafman changed the title test: Use egrep variant to be more generic and update some patterns test: Use grep -E variant to be more generic and update some patterns Sep 8, 2018
@dzafman
Copy link
Contributor Author

dzafman commented Sep 8, 2018

Testing PASSED
http://pulpito.ceph.com/dzafman-2018-09-07_18:07:43-rados-wip-35845-distro-basic-smithi/

The problem was seen on Mimic, but this passes master on the same distribution Ubuntu 16.04

@dzafman dzafman requested a review from jdurgin September 8, 2018 02:07
@dzafman
Copy link
Contributor Author

dzafman commented Sep 8, 2018

Now I'm not sure why this is passing on my build machine which is Ubuntu 16.04 but not on a teuthology test machine unless my machine needs updating or someone is picking up a different version of grep.

@liewegas
Copy link
Member

retest this please

@dzafman
Copy link
Contributor Author

dzafman commented Sep 10, 2018

I found the real problem will close and open a different pull request.

@dzafman dzafman closed this Sep 10, 2018
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