Skip to content

Conversation

@deven0t
Copy link
Contributor

@deven0t deven0t commented Feb 13, 2023

In case of RKE, env error comes with exit status 1, so added OR codition to match with error text as well.

resolve: #1364

In case of RKE, env error comes with exit status 1, so added OR codition to match with error text as well.

resolve: aquasecurity#1364
@deven0t
Copy link
Contributor Author

deven0t commented Feb 13, 2023

Hi @chen-keinan can you please review it.
Thanks

@chen-keinan
Copy link
Contributor

lgtm

@deven0t
Copy link
Contributor Author

deven0t commented Feb 14, 2023

@mozillazg please review the changes.
Thanks

Copy link
Contributor

@mozillazg mozillazg left a comment

Choose a reason for hiding this comment

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

LGTM

@chen-keinan chen-keinan merged commit fc72a8a into aquasecurity:main Feb 14, 2023
@deven0t deven0t deleted the deven-bugfix-file-not-found branch February 15, 2023 07:26
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.

False negative when audit_config is defined and config file is not found.

3 participants