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.

Sign up for GitHub

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

Jump to bottom

[META 690] Ensure a minimum retry interval of 5 seconds in fetching central configuration #2941

Closed
elastic-apm-tech opened this issue Sep 23, 2022 · 0 comments · Fixed by #3008
Closed

[META 690] Ensure a minimum retry interval of 5 seconds in fetching central configuration #2941

elastic-apm-tech opened this issue Sep 23, 2022 · 0 comments · Fixed by #3008
Assignees
@trentm
Labels
agent-nodejs Make available for APM Agents project planning.
Milestone
8.6

Comments

@elastic-apm-tech
Copy link

See meta issue for the description and details:
@elastic-apm-tech elastic-apm-tech mentioned this issue Sep 23, 2022
Open
6 tasks
@github-actions github-actions bot added the agent-nodejs Make available for APM Agents project planning. label Sep 23, 2022
@estolfo estolfo added this to the 8.6 milestone Sep 27, 2022
@trentm trentm self-assigned this Oct 24, 2022
trentm added a commit to elastic/apm-nodejs-http-client that referenced this issue Oct 24, 2022
@trentm
fix: put [5s, 1d] range guard on the delay before re-fetching central…
f29a7d0 
… config

Refs: elastic/apm-agent-nodejs#2941
@trentm trentm mentioned this issue Oct 24, 2022
Merged
trentm added a commit to elastic/apm-nodejs-http-client that referenced this issue Oct 25, 2022
@trentm
fix: put [5s, 1d] range guard on the delay before re-fetching central…
6c8d141 
… config (#185)

Refs: elastic/apm-agent-nodejs#2941
trentm added a commit that referenced this issue Oct 27, 2022
@trentm
feat: container-info for AWS ECS/Fargate; [5s,1d] guard on central co…
0dd6260 
…nfig re-fetch interval

Update elastic-apm-http-client to get these changes.
This also drops using 'container-info' directly in this repo's tests.
Given that the impl in that module and in the http-client might
diverge over time, it isn't necessarily valid to use 'container-info'
here.

Closes: #2914
Closes: #2941
@trentm trentm mentioned this issue Oct 27, 2022
Merged
@trentm trentm closed this as completed in 350e750 Oct 27, 2022
@trentm trentm mentioned this issue Nov 15, 2022
Closed
PeterEinberger pushed a commit to fpm-git/apm-agent-nodejs that referenced this issue Aug 20, 2024
@trentm @PeterEinberger
feat: container-info for AWS ECS/Fargate; [5s,1d] guard on central co…
01bc3c8 
…nfig re-fetch interval (elastic#3008)

Update elastic-apm-http-client to get these changes.
This also drops using 'container-info' directly in this repo's tests.
Given that the impl in that module and in the http-client might
diverge over time, it isn't necessarily valid to use 'container-info'
here.

Closes: elastic#2914
Closes: elastic#2941
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@trentm trentm

Labels
agent-nodejs Make available for APM Agents project planning.
Projects
None yet
Milestone
8.6
3 participants
@trentm @estolfo @elastic-apm-tech