Tags: commercetools/telefonistka
Tags
Improve application diffs (#23) * Improve application diff This changes the application diff to render a YAML diff with a number of context lines instead of relying on the cmp library previously used which would render a diff of Go objects. * Allow bare URL in markdown After discussion it has been deemed unnecessary to enforce non-bare URLs in markdown. * Upgrade golangci-lint The earlier version fails to run on Darwin with Go 1.23.
Sd 607 provide the compelete name of target cluster in promotion prs (#… …21) * Configure mirrord to 'steal' traffic. Stealing traffic means the Telefonistka instance running in the cluster will not receive the webhook, only the local instance will. This is so that we only see results from our local instance. * Change format of promotion path comment. The promotion path used to look like 'source/'->['path1/','path2/']. This PR changes it to look like: 'source/'-> 'path1/' 'path2/' * Remove truncation of sp in promotionPRBody * Add test for prBody * Run gofumpt * Add md extension to golden file * Addjusted prBody and matching test/goldenfile * Remove dead code
Provide Diff for new apps (#18) * Initial commit of "Provide Diff for new apps" * Delete temp app after diff * Nest ArgoCD configuration keys under one key. * Add configuration key to toggle this feature * Add some feature limitation to docs * Use prBtanch for temp app object TargetRevision
Allow selectively redacting diff content (#19) * Add a new component-level "disableDiff" configuration key * Read the configuration file before diff * Pass the the config the relevant function * Replace diff content with a relevant message * Document new configuration key Co-authored-by: Yazdan Mohammadi <[email protected]> --------- Co-authored-by: Yazdan Mohammadi <[email protected]>
Allow setting argoCD revision to PR git Branch (#16) * Allow setting argoCD revision to PR git branch * Triggering from checkbox event was written in a generic way for future proofing * Document new config key --------- Co-authored-by: Hannes Gustafsson <[email protected]>
Allow setting argoCD revision to PR git Branch (#16) * Allow setting argoCD revision to PR git branch * Triggering from checkbox event was written in a generic way for future proofing * Document new config key --------- Co-authored-by: Hannes Gustafsson <[email protected]>
Handle/mitigate crash and context issues (#11) * Use new context (not a child) * Remove calls to a function that exits the process. * Add more "context"(info, like for humans) to errors * Address another err with no context issue * Respond the webhook only based on payload parsing. Actual Event processing is move to background thread. This require some refactoring, created ReciveWebhook and ReciveEventFile functions to represent the different behavior in Web Server VS CLI triggering while keeping to the GH stuff in the GH package * Cancel whole drift work on context deadline * Move error function return value to the standard position Handle cases where GetContents returns nil HTTP response (like in Context cancellation)
Fix relevant components discovery and add support for ArgoCD "native"… … application discovery (#8) * Use telefonistka in-pr metadata mechanism to pass changed component path to ArgoCD diff functionality * Add some validation to avoid auto-merging PRs in cases telefonistka is unsure of the component that where changed * Support two way to fetch matching ArgoCD apps from a Telefonistka component the telefonistka dedicated sha1 label and the ArgoCD native `manifest-generate-paths` annotation * Fix relevant components discovery hannes (#9) * mocks: use go run to reduce dependency Using go run will automatically install the tool and run it if it is not installed already. * argocd: fix import path The import path for the mocks package is incorrect causing errors. The mocks package should be imported from internal/pkg/mocks instead of mocks/argocd. This also removes the aliased import and uses the package as named, which aligns with the directory name. --------- Co-authored-by: Hannes Gustafsson <[email protected]> * Add needed argodiff.NewDiffConfigBuilder param (after version upgrade) Log time of API call with questionable performance :) * Fix typo and add code generate step to makefile * Lint fixes, fix gitignore. upsate go.mod * Hack go genrate for linting * Cleanup and try to use make file config * Document new configuration key * Update argocd_test.go - remove some editor completion artifacts * Fix two "bad conflict resolution" issues * Apply suggestions from code review Co-authored-by: Yazdan Mohammadi <[email protected]> * I fix var name typo * Rename client var names --------- Co-authored-by: Hannes Gustafsson <[email protected]> Co-authored-by: Yazdan Mohammadi <[email protected]>