Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add k8s context #5

Open
dcramer opened this issue Dec 20, 2019 · 1 comment
Open

Add k8s context #5

dcramer opened this issue Dec 20, 2019 · 1 comment
Labels
help wanted Extra attention is needed question Further information is requested

Comments

@dcramer
Copy link

dcramer commented Dec 20, 2019

Something I'm looking to evaluate and contribute back (in hopes that maybe we can standardize on a more correct/performant/useful SDK), is adding additional context/index capabilities.

Contexts look like this:

{
  // event payload
  "contexts": {
    "kubernetes": {
       "key": "value"
    }
  }
}

Eventually these will get auto-indexed like tags do, but right now we still have to send both tags (for search) and contexts for display. It's basically a more flexible version of 'extra' (since its just a namespaced extra).

I think for starters it'd be good to take every tag or generic object metadata and stuff it into the context.

@wichert
Copy link
Owner

wichert commented Mar 25, 2020

That should be pretty easy to do. I already include labels as tags; is there much useful value in including them in the context as well?

@wichert wichert added help wanted Extra attention is needed question Further information is requested labels Oct 5, 2020
janLo pushed a commit to janLo/k8s-sentry that referenced this issue Dec 16, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants