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

[META 321] Add log_level spec #1826

Closed
basepi opened this issue Oct 5, 2020 · 7 comments · Fixed by #1908
Closed

[META 321] Add log_level spec #1826

basepi opened this issue Oct 5, 2020 · 7 comments · Fixed by #1908
Assignees
Labels
agent-nodejs Make available for APM Agents project planning. agent-spec Related to a cross agent functionality spec. cross APM agents
Milestone

Comments

@basepi
Copy link

basepi commented Oct 5, 2020

Implementing elastic/apm#332

Note that the implementation of this spec also includes adding the option to Kibana.

@astorm astorm added the agent-spec Related to a cross agent functionality spec. label Oct 8, 2020
@trentm trentm self-assigned this Nov 4, 2020
@AlexanderWert AlexanderWert added this to the 7.11 milestone Nov 12, 2020
@AlexanderWert AlexanderWert added cross APM agents agent-nodejs Make available for APM Agents project planning. labels Nov 12, 2020
@AlexanderWert AlexanderWert changed the title Add log_level spec [META 321] Add log_level spec Nov 25, 2020
@trentm trentm linked a pull request Dec 8, 2020 that will close this issue
trentm added a commit that referenced this issue Dec 8, 2020
@zube zube bot added [zube]: Done and removed [zube]: Ready labels Dec 8, 2020
@trentm
Copy link
Member

trentm commented Dec 8, 2020

That commit doesn't fully resolve this ticket: the kibana.git config update remains.
Rhetorical Qs: Why does GitHub thinks this closes the ticket? Is that implicit in having added this issue to "Linked issues" for the PR?

@trentm trentm reopened this Dec 8, 2020
@zube zube bot added [zube]: Inbox and removed [zube]: Done labels Dec 8, 2020
@basepi
Copy link
Author

basepi commented Dec 8, 2020

@trentm I've only ever linked issues by mentioning them. Perhaps if you add them in the sidebar it assumes you want to close them when the PR is merged.

If I just want to like them without automatic closing, I do:

Refs #1826

If I want the issue to close when the PR is merged, I do:

Closes #1826

(There are a few keywords that will result in a closed issue: fix, fixes, close, closes, resolves to name a few)

@trentm
Copy link
Member

trentm commented Dec 8, 2020

@basepi Thanks. Yah, I've been using "Fixes: #NNN" in the commit message to close.

From https://docs.github.com/en/free-pro-team@latest/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue

When you merge a linked pull request into the default branch of a repository, its linked issue is automatically closed.

So I think I'll stick to just using the commit message/initial PR description, and I'll try to pick up using "Refs ..." syntax.

@basepi
Copy link
Author

basepi commented Dec 8, 2020

Yeah as long as you stay away from any of those close keywords, you should be fine.

@trentm
Copy link
Member

trentm commented Dec 8, 2020

elastic/kibana#85346 is for the Kibana related change.

@trentm
Copy link
Member

trentm commented Dec 9, 2020

Kibana master: elastic/kibana#85346
Kibana 7.x backport: elastic/kibana#85460

@trentm
Copy link
Member

trentm commented Dec 9, 2020

Those are merged now.

@trentm trentm closed this as completed Dec 9, 2020
@zube zube bot added [zube]: Done and removed [zube]: Inbox labels Dec 9, 2020
@zube zube bot removed the [zube]: Done label Mar 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agent-nodejs Make available for APM Agents project planning. agent-spec Related to a cross agent functionality spec. cross APM agents
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants