Skip to content

[META 27] Align ecs-logging-python #26

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
Closed
4 tasks done
basepi opened this issue Dec 3, 2020 · 4 comments · Fixed by #31
Closed
4 tasks done

[META 27] Align ecs-logging-python #26

basepi opened this issue Dec 3, 2020 · 4 comments · Fixed by #31
Assignees
@basepi
Labels
agent-python
Milestone
7.11

Comments

@basepi
Copy link
Contributor

basepi commented Dec 3, 2020 •
edited
Loading

See elastic/ecs-logging#27 for details on alignment.
@AlexanderWert AlexanderWert mentioned this issue Dec 16, 2020
Closed
@AlexanderWert AlexanderWert added this to the 7.11 milestone Dec 21, 2020
@beniwohli
Copy link

@sethmlarson @basepi as far as I can tell, what's missing from our side is some automated validation of the spec. Is this something anyone is particularly keen to implement? :D

@basepi
Copy link
Contributor Author

basepi commented Jan 7, 2021

Yeah it's on my list. Not super keen but it needs to be done. :)

@basepi basepi self-assigned this Jan 7, 2021
@sethmlarson
Copy link
Contributor

Btw @basepi I have a local branch which gets the ordering of fields correct (@timestamp, log.level, and message) will be creating the PR today.

@sethmlarson sethmlarson mentioned this issue Jan 7, 2021
Merged
This was referenced Jan 12, 2021
Merged
Merged
@basepi
Copy link
Contributor Author

basepi commented Feb 9, 2021

Ref: https://github.com/elastic/open-source/issues/194

@basepi basepi mentioned this issue Feb 9, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@basepi basepi

Labels
agent-python
Projects
None yet
Milestone
7.11
Development

Successfully merging a pull request may close this issue.

Validate against spec basepi/ecs-logging-python
4 participants
@beniwohli @basepi @AlexanderWert @sethmlarson