New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make second group match of ufw status output optional #56678
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! Thanks!
The tests are failling; I want to investigate this a bit, something looks fishy.
You need to add a changelog fragment: https://docs.ansible.com/ansible/latest/reference_appendices/release_and_maintenance.html#changelogs-how-to (see here for examples).
Found the problem; the comparison logic was kind of broken, I've pushed a fix for that into your branch. That should fix the tests. Do you want to add the changelog, or do you want me to do that? |
Nice. I will go ahead and add it
…On Mon, May 20, 2019, 10:43 PM Felix Fontein ***@***.***> wrote:
Found the problem; the comparison logic was kind of broken, I've pushed a
fix for that into your branch. That should fix the tests. Do you want to
add the changelog, or do you want me to do that?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#56678?email_source=notifications&email_token=AAQVQQTNZ52PDR23QFFBL2LPWODZVA5CNFSM4HOGA7H2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV2Z6EY#issuecomment-494247699>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAQVQQUNNSQB36QRREY3XQLPWODZVANCNFSM4HOGA7HQ>
.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@ovcharenko @jbarotin anyone wants to take a look at this? |
LGTM |
shipit |
* Make second group match of ufw status output optional Fixes ansible#56674 * Fix comparison logic. * Add changelog fragment (cherry picked from commit 1d22909)
Fixes #56674
SUMMARY
Second group match does not always exist. In cases where it is missing, like when logging is set to
off
, allow regex to still succeed to return first group match such thatcurrent_on_off_value
will be defined.ISSUE TYPE
COMPONENT NAME
ufw