Skip to content

Log when next retry request to Knapsack Pro API happens before starting Fallback Mode#114

Merged
ArturT merged 2 commits intomasterfrom
log-when-next-retry-request
Jun 2, 2020
Merged

Log when next retry request to Knapsack Pro API happens before starting Fallback Mode#114
ArturT merged 2 commits intomasterfrom
log-when-next-retry-request

Conversation

@ArturT
Copy link
Copy Markdown
Member

@ArturT ArturT commented Jun 2, 2020

change

This adds log warning message: Next request in 8s... etc to knapsack pro logger to notify the user when next retry request to Knapsack Pro API will happen.

why

CodeShip seems to freeze when no output for a long time so we print some output to show that knapsack_pro process is working.

Example output

4 #50431]  WARN -- : [knapsack_pro] #<SocketError: Failed to open TCP connection to api-fake.knapsackpro.test:3000 (getaddrinfo: nodename nor servname provided, or not known)>
W, [2020-06-02T09:28:04.961195 #50431]  WARN -- : [knapsack_pro] Wait 8s and retry request to Knapsack Pro API.
W, [2020-06-02T09:28:04.961239 #50431]  WARN -- : [knapsack_pro] Next request in 8s...
W, [2020-06-02T09:28:06.961402 #50431]  WARN -- : [knapsack_pro] Next request in 6s...
W, [2020-06-02T09:28:08.966389 #50431]  WARN -- : [knapsack_pro] Next request in 4s...
W, [2020-06-02T09:28:10.969894 #50431]  WARN -- : [knapsack_pro] Next request in 2s...
W, [2020-06-02T09:28:12.973823 #50431]  WARN -- : [knapsack_pro] #<SocketError: Failed to open TCP connection to api-fake.knapsackpro.test:3000 (getaddrinfo: nodename nor servname provided, or not known)>
W, [2020-06-02T09:28:12.973963 #50431]  WARN -- : [knapsack_pro] Wait 16s and retry request to Knapsack Pro API.
W, [2020-06-02T09:28:12.974035 #50431]  WARN -- : [knapsack_pro] Next request in 16s...
W, [2020-06-02T09:28:14.975409 #50431]  WARN -- : [knapsack_pro] Next request in 14s...
W, [2020-06-02T09:28:16.976070 #50431]  WARN -- : [knapsack_pro] Next request in 12s...
W, [2020-06-02T09:28:18.976941 #50431]  WARN -- : [knapsack_pro] Next request in 10s...
W, [2020-06-02T09:28:20.980815 #50431]  WARN -- : [knapsack_pro] Next request in 8s...
W, [2020-06-02T09:28:22.985190 #50431]  WARN -- : [knapsack_pro] Next request in 6s...
W, [2020-06-02T09:28:24.985996 #50431]  WARN -- : [knapsack_pro] Next request in 4s...
W, [2020-06-02T09:28:26.991360 #50431]  WARN -- : [knapsack_pro] Next request in 2s...

@ArturT ArturT merged commit 122fa36 into master Jun 2, 2020
@ArturT ArturT deleted the log-when-next-retry-request branch June 2, 2020 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant