Skip to content

Tags: commercetools/telefonistka

Tags

v1.19.0

Toggle v1.19.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
fix recviveEventFile function paramater ordering issue (#168)

* fix recviveEventFile function paramater ordering issue

Signed-off-by: Oded Ben Ozer <oded.benozer@gmail.com>

---------

Signed-off-by: Oded Ben Ozer <oded.benozer@gmail.com>

v1.17.3

Toggle v1.17.3's commit message

Verified

This commit was signed with the committer’s verified signature.
hnnsgstfssn Hannes Gustafsson
Fix comment event guard (#118)

The commit [1] incorrectly passes issue.user.login as owner when fetching the
PR. This causes events to be considered incorrectly.

Revert back to using details.Repo and details.Owner directly and leave
further refactoring for later.

[1] 79026b0

v1.18.1

Toggle v1.18.1's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Fix comment event guard (#118)

The commit [1] incorrectly passes issue.user.login as owner when fetching the
PR. This causes events to be considered incorrectly.

Revert back to using details.Repo and details.Owner directly and leave
further refactoring for later.

[1] 79026b0

v1.18.0

Toggle v1.18.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Add support for retriggering via PR comment (#115)

* Add support for retriggering via PR comment

* fixup! Add support for retriggering via PR comment

* fixup! Add support for retriggering via PR comment

* Make handleChangedPREvent able to handle comment event

Instead of passing in the PullRequestEvent, the signature is changed to
accept the only used attributes of PR number and the list of labels.

This allows it to be called when receiving a IssueCommentEvent,
triggered from a "retrigger"-comment.

* Move panic handler

A triggering comment can now potentially cause a panic so the handler is
moved to cover such cases, avoiding a crashed server if there are bugs.

* Refactor getting PR ref and SHA

The only use of GetRef and GetSHA is in the comment event handler.

This refactors its use in an effort to simplify; the comment handler
instead fetches the PR object directly once, and aborts if the issue is
not linked to a PR.

It then sets the ref and SHA based on the returned object values.

* Trim off leading and trailing space

* Lint

* Log when bailing

v1.17.2

Toggle v1.17.2's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Revert go.mod to v1.15.0 version (#101)

* Revert go.mod to v1.15.0 version

* Go mod tidy again and now

* go version 1.24.1

* Resolve lint errors

---------

Co-authored-by: Yazdan Mohammadi <yazdan.mohammadi@commercetools.com>

v1.15.0

Toggle v1.15.0's commit message

Partially verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
Generate argoCD diff comments with Go code instead of GoTemplate (#72)

* Initial commit for avoiding goTemplate in PR comment
* Avoid displaying health/status warnings for "temp" apps.
* Add tests
* Tune comment size test limit to match new data in diff_comment_data_test.json"
* Avoid linting the testdata markdown file (for now)

Co-authored-by: Hannes Gustafsson <hannes.gustafsson@commercetools.com>

v1.14.0

Toggle v1.14.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Warn user when changing ArgoCD apps that have sync/health issues (#65)

* Warn user when changing ArgoCD apps that are unhealthy/OutOfSync or
doesn't have auto-sync on

* Ensure comment length test doesn't include health/sync warnings

v1.13.0

Toggle v1.13.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Support custom generator (#46)

* argocd: guard against missing Git configuration

If the found ApplicationSet contains no Git configuration Telefonistka
crashes.

Guarding against the missing configuration should avoid the crash.

* Make test parallel

* Use component path that works in lab

* Support custom generator

Adds support for custom generator as long as the generator has an input
parameter named "path".

Note that it would probably be advisable to make it configurable, but
this is left as a future exercise for the reader.

v1.12.0

Toggle v1.12.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Add logging around generating diff comment (#44)

v1.11.0

Toggle v1.11.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
monitoring PR handling failures. (#42)

Instrument  PR handling failures.

We track both explicit failures with pr_handle_failures_total
And cases where telefonistka commit status check  is left in "pending"
state (because Telefonistka exploded while handling that event )


Co-authored-by: Yazdan Mohammadi <yazdan.mohammadi@commercetools.com>