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

Add support for Google Chat alerts #953

Merged
merged 8 commits into from
Aug 24, 2021
Merged

Conversation

fpinna
Copy link
Contributor

@fpinna fpinna commented Jul 9, 2021

Add google chat alerting support

fpinna added 4 commits July 9, 2021 17:44
Signed-off-by: fpinna <fplkid@gmail.com>
Signed-off-by: fpinna <fplkid@gmail.com>
Signed-off-by: fpinna <fplkid@gmail.com>
Signed-off-by: fpinna <fplkid@gmail.com>
@fpinna fpinna requested a review from stefanprodan as a code owner July 9, 2021 21:03
@hiddeco
Copy link
Member

hiddeco commented Jul 9, 2021

Just want to give you a heads up that @stefanprodan is out on holidays and this PR probably won't be reviewed before he is back (in two weeks).

@fpinna
Copy link
Contributor Author

fpinna commented Jul 12, 2021

Just want to give you a heads up that @stefanprodan is out on holidays and this PR probably won't be reviewed before he is back (in two weeks).

Thank you for your feedback.

Signed-off-by: fpinna <fplkid@gmail.com>
@stefanprodan
Copy link
Member

Was this tested? I really doubt it works since the gchat is missing from the Notifier Factory.

Signed-off-by: fpinna <fplkid@gmail.com>
@fpinna
Copy link
Contributor Author

fpinna commented Jul 28, 2021

Was this tested? I really doubt it works since the gchat is missing from the Notifier Factory.

Sorry, my bad. I made mistakes with my last commit.

@@ -1191,6 +1191,7 @@ spec:
- msteams
- discord
- rocket
- gchat
Copy link
Member

Choose a reason for hiding this comment

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

Please add this to artifacts/flagger/crd.yaml then run make crd and commit all changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Signed-off-by: fpinna <fplkid@gmail.com>
@@ -52,6 +52,8 @@ func (f Factory) Notifier(provider string) (Interface, error) {
n, err = NewRocket(f.URL, f.ProxyURL, f.Username, f.Channel)
case "msteams":
n, err = NewMSTeams(f.URL, f.ProxyURL)
case "gchat":
n, err = NewMSTeams(f.URL, f.ProxyURL)
Copy link
Member

Choose a reason for hiding this comment

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

This should be NewGChat

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.
thanks

@codecov-commenter
Copy link

codecov-commenter commented Aug 23, 2021

Codecov Report

Merging #953 (ce4cd61) into main (7c92b33) will increase coverage by 0.05%.
The diff coverage is 84.21%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #953      +/-   ##
==========================================
+ Coverage   56.91%   56.97%   +0.05%     
==========================================
  Files          71       73       +2     
  Lines        5861     5924      +63     
==========================================
+ Hits         3336     3375      +39     
- Misses       2030     2048      +18     
- Partials      495      501       +6     
Impacted Files Coverage Δ
pkg/notifier/factory.go 0.00% <0.00%> (ø)
pkg/notifier/gchat.go 88.88% <88.88%> (ø)
pkg/router/factory.go 0.00% <0.00%> (ø)
pkg/loadtester/concord.go 29.03% <0.00%> (ø)
pkg/metrics/observers/factory.go 0.00% <0.00%> (ø)
pkg/metrics/observers/osm.go 46.66% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7c92b33...ce4cd61. Read the comment docs.

Signed-off-by: fpinna <fplkid@gmail.com>
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @fpinna

@stefanprodan stefanprodan changed the title Google chat Add support for Google Chat alerts Aug 24, 2021
@stefanprodan stefanprodan merged commit cf8783e into fluxcd:main Aug 24, 2021
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.

4 participants