-
Notifications
You must be signed in to change notification settings - Fork 2.4k
jira integration #3590
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
jira integration #3590
Changes from all commits
ab5c92c
cf3de82
d236da4
e22d06b
bf991e0
6fb2bd1
c2ceba2
a0b94df
3705a64
25b2657
d65ca8b
a3975d9
74138ba
ed70001
eb64057
8874392
3aa5a4c
9604737
6d2ceba
f945810
3229180
27c8b44
aac35bd
81d66d9
ebe6269
cdc96e6
b15de33
bd5b579
ebf5923
25808bf
3e35db2
3166f69
495e21f
6e88b9d
a7917b3
48a904b
510ccb6
23628b3
ca08adb
dbda07a
b6df3c1
8bb43d0
e123003
0ac2ca5
43c8f10
3517487
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -37,6 +37,7 @@ require ( | |
| github.com/shurcooL/httpfs v0.0.0-20190707220628-8d4bc4ba7749 | ||
| github.com/shurcooL/vfsgen v0.0.0-20200824052919-0d455de96546 | ||
| github.com/stretchr/testify v1.9.0 | ||
| github.com/trivago/tgo v1.0.7 | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. A new dependency is mandatory for the map[any]any cast, because
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not terribly excited to depend on a project that isn't very active (e.g. it doesn't have a go.mod). While I understand that it is stable and proven, I'll look for other options. Can you also provide an example of something that breaks encoding/json?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Alertmanager parses the nested map as
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
https://github.com/json-iterator/go natively supports encoding of If you try to go with own code, I have prepare a test file for you: https://github.com/jkroepke/homelab/blob/main/go/map_cast/main_test.go I also figured out that Not 100% sure about this, but I guess, the issue could be also resolved, if
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. json-iterator is already an indirect dependency via prometheus/client_golang so I'd say that it's ok to use it.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I have no clue, why json-iterator works fine on the lined go test above, but while integrate it inside AM, tests in AM are failed with I may give up here, since I don't know why it works on a synthetic environment, but not on AM. For reference, here is my code (cf85bbe) |
||
| github.com/xlab/treeprint v1.2.0 | ||
| go.uber.org/atomic v1.11.0 | ||
| go.uber.org/automaxprocs v1.5.3 | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.