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

Ensure that the running agent's config is used when restarted #763

Merged
merged 2 commits into from
Apr 14, 2020

Conversation

estolfo
Copy link
Contributor

@estolfo estolfo commented Apr 14, 2020

This fixes a bug with the ElasticAPM.restart method: if the method is called with no config passed in, the previously running agent's config will not be used, though it should be.

Closes #764

@estolfo estolfo modified the milestone: 7.7 Apr 14, 2020
@estolfo estolfo merged commit b819708 into elastic:master Apr 14, 2020
v1v added a commit to v1v/apm-agent-ruby that referenced this pull request Apr 21, 2020
…release

* upstream/master: (24 commits)
  Ensure that a new central config fetch is scheduled after network error (elastic#772)
  Updates to Resque support and documentation (elastic#768)
  Support recording config option (elastic#765)
  Add tests and adjustments to classes that can be restarted (elastic#766)
  Ensure that the running agent's config is used when restarted (elastic#763)
  The enabled tag is config-enabled
  Add enabled config option replacing active (elastic#762)
  docs: add dynamic badge (elastic#760)
  Config values set to false should be false, not nil (elastic#761)
  Add changelog entry for updating log level on config's logger
  Add changelog entry for logging updated remote config values
  Add changelog entry for dynamic config values
  Check if env variable exists before setting up SimpleCov (elastic#759)
  Log the updated options from central config in addition to cumulative modifications (elastic#758)
  ci(jenkins): merge branch in downstream jobs (elastic#757)
  Log level on logger is updated when Config options are (elastic#755)
  Cobertura coverage (elastic#736)
  Allow Config values to be dynamic (elastic#747)
  Reorder checks in test in case values are being cleared before checked (elastic#743)
  ci(jenkins): disable stages for only docs (elastic#745)
  ...
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.

The ElasticAPM.restart method doesn't reuse the running agent's config
1 participant