Skip to content

Updates for verification#31

Merged
philip-loggly merged 3 commits intomasterfrom
add-tag
Sep 2, 2013
Merged

Updates for verification#31
philip-loggly merged 3 commits intomasterfrom
add-tag

Conversation

@ivangonekrazy
Copy link
Copy Markdown
Contributor

  • extend the verification loop timeout
  • adds an 'example' tag to the syslog-ng/rsyslogd templates to workaround https://loggly.jira.com/browse/LOGGLY-3884
  • adds some more feedback while we're polling for the verification event

♥ Ivan Tam ♥ added 2 commits August 31, 2013 02:34
Also adds more feedback during the verify loop.
@ivangonekrazy
Copy link
Copy Markdown
Contributor Author

@rhigdon @jimniz please review and merge if we wanna pull in the workaround for the APIv2 no-tag issue.

@philip-loggly
Copy link
Copy Markdown
Contributor

Seems like we have no choice but to add this "example" tag to the template. Is that correct?

@ivangonekrazy
Copy link
Copy Markdown
Contributor Author

In the short term, yes.

Otherwise the API call to do the verification fails. There's alreadyva fix
for the api endpoint. We just need to deploy it after GA
On Sep 1, 2013 6:27 PM, "philip-loggly" notifications@github.com wrote:

Seems like we have no choice but to add this "example" tag to the
template. Is that correct?


Reply to this email directly or view it on GitHubhttps://github.com//pull/31#issuecomment-23637256
.

@philip-loggly
Copy link
Copy Markdown
Contributor

Why make the timeout so long? It seems excessive.

philip-loggly added a commit that referenced this pull request Sep 2, 2013
Updates for verification
@philip-loggly philip-loggly merged commit d0f35ef into master Sep 2, 2013
@philip-loggly
Copy link
Copy Markdown
Contributor

Merged, though I would like to know why the timeout is so long.

@ivangonekrazy
Copy link
Copy Markdown
Contributor Author

The previous timeout length was too short and verifications ended up always timing out.

I wanted to err on the side of letting the script try for longer rather than generating more emails for support, at least for the short-term until we figure out the fate of the current script.

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.

2 participants