Skip to content

Add configuration option skywalking_agent.runtime_dir.#47

Merged
wu-sheng merged 2 commits intoapache:masterfrom
jmjoy:sock
Jan 20, 2023
Merged

Add configuration option skywalking_agent.runtime_dir.#47
wu-sheng merged 2 commits intoapache:masterfrom
jmjoy:sock

Conversation

@jmjoy
Copy link
Member

@jmjoy jmjoy commented Jan 20, 2023

  1. Add configuration option skywalking_agent.runtime_dir, indicate the runtime directory for holding sock files now.
  2. Update the document about option.

@jmjoy jmjoy added this to the 0.3.0 milestone Jan 20, 2023
@jmjoy jmjoy marked this pull request as ready for review January 20, 2023 06:45
@jmjoy jmjoy requested review from heyanlong and wu-sheng January 20, 2023 06:45

```ini
[skywalking]
[skywalking_agent]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, we change from skywalking to skywalking_agent now?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This tag isn't useful for php, but ours extension name is skywalking_agent, so skywalking_agent is more accurate.

@wu-sheng wu-sheng merged commit 9d2d05a into apache:master Jan 20, 2023
@jmjoy jmjoy deleted the sock branch January 20, 2023 06:54
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