Skip to content
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

LOG-6073 Add default value for count search #128

Merged
merged 1 commit into from Mar 3, 2020
Merged

LOG-6073 Add default value for count search #128

merged 1 commit into from Mar 3, 2020

Conversation

@jirka-kruml
Copy link
Contributor

@jirka-kruml jirka-kruml commented Mar 3, 2020

Resolves CI https://swicloud.atlassian.net/browse/LOG-7606
Just adds validation that count, which was fetched from our API, returns a valid number, if not, it means that no event were found, so set the value to 0. Otherwise the script prints configure-linux.sh: line 902: [: : integer expression expected which is propagated further and the output doesn't look clean.

@jirka-kruml jirka-kruml requested a review from loggly/lg1-devs Mar 3, 2020
@jirka-kruml jirka-kruml self-assigned this Mar 3, 2020
@Katulus
Katulus approved these changes Mar 3, 2020
@jirka-kruml jirka-kruml merged commit 515212e into master Mar 3, 2020
@jirka-kruml jirka-kruml deleted the jk/LOG-7606 branch Mar 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.