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

Revert "LOG-6073 Add default value for count search" #130

Merged
merged 1 commit into from Mar 6, 2020
Merged

Conversation

@jirka-kruml
Copy link
Contributor

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

This reverts commit e05e977.

According to Jarvis:
Hi Jiri, I just wanted to inform you that it seems like the automatic script is not working properly. It is stuck at this area of the code

logMsgToConfigSysLog "INFO" "INFO: Verifying if the logs made it to Loggly."
logMsgToConfigSysLog "INFO" "INFO: Verification # $counter of total $maxCounter."
#get the final count of file logs for past 15 minutes
searchAndFetch fileLatestLogCount "$queryUrl"
let counter=$counter+1

I've tested this on two environment, one EC2 Linux 2 and Ubuntu image in AWS, both are stuck at the verification part

This happens here

@jirka-kruml jirka-kruml requested a review from loggly/lg1-devs Mar 5, 2020
@jirka-kruml jirka-kruml self-assigned this Mar 5, 2020
@Katulus
Katulus approved these changes Mar 6, 2020
@jirka-kruml jirka-kruml merged commit 749669c into master Mar 6, 2020
@jirka-kruml jirka-kruml deleted the jk/LOG-6073 branch Mar 6, 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.