feat: cd pipeline deployment history refactoring #5200
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kartik-579
requested review from
vikramdevtron,
kripanshdevtron and
nishant-d
as code owners
kartik-579
changed the title
github-actions
bot
added
PR:Ready-to-Review
and removed
PR:Issue-verification-failed
labels
vikramdevtron
approved these changes
Jun 6, 2024
|
Ash-exp
added a commit
that referenced
this pull request
Jun 10, 2024
* feat: replace user_deployment_request.status with pipeline_status_timelines * updated schema migration script * fix: migration script * update readme to include multi arch flag (#4998) * updated trigger event func and timeline names * review comment incorporation * fix: GetAllInCompleteRequests query * fix: GetAllInCompleteRequests query * fixed: UpdatePreviousQueuedRunnerStatus error * fix: loogers * doc: Added enhancements to security doc (#5203) * Rough Draft Initiated * Created Security Enhancement Docs + Other Adhoc Fixes * Incorporated PM Feedback * Modified Portforward Section to Kubectl Section (#5236) * feat: cd pipeline deployment history refactoring (#5200) * searchableKey service move * migrated cd deployment history code * idempotency of triggerPipeline func and minor fixes * timeline fixes * updated: NewTriggerEvent func * updated ErrorDeploymentSuperseded err handling * updated: GetArgoPipelinesHavingLatestTriggerStuckInNonTerminalStatuses statuses * fix: timeline status * refactored: UpdatePreviousQueuedRunnerStatus * fix: GetPreviousQueuedRunners query * fix: UpdateRunnerStatusToFailedForIds query * fix: buildTriggerEventForOverrideRequest func * fix: UpdateRunnerStatusToFailedForIds query * fix: UpdateRunnerStatusToFailedForIds query * updated: handleAsyncTriggerReleaseError func * fix: concurrent case handlings * updated folder name * updated GetLastStatusPublishedTimeForWfrId func * feat: removed audit timeline and error handlings * fix: FetchTimelinesForWfrIdExcludingStatuses query * fix: deleted pipeline flow * timeline handling * timeline handling fixed * removed unused const * chore: updated go-bitbucket version * removed unnecessary alais --------- Co-authored-by: ayu-devtron <[email protected]> Co-authored-by: ashokdevtron <[email protected]> Co-authored-by: kartik-579 <[email protected]>
Ash-exp
added a commit
that referenced
this pull request
Jun 19, 2024
* chore: context refactoring for gitops async * chore: refactored context for argo app deploy operations * tracer refactoring wip * fixed: timeline status update issue and added force sync param - WIP * fixed: undeclared varibale * env gen updated * common-lib version updated * async deployment gitops and CEL expression * chore: cel library added * chore: go mod tidy * feat: Async gitops deployment flow * updated trigger pipeline scope * triggerPipeline refactoring * runner status update * udpated pipelineOverride object in GetValuesOverrideForTrigger * updated UserDeploymentRequest status * cron method: incomplete deployment request * updated alias naming * updated alias naming * updated go-bitbucket librabry * added github.com/ash-exp/go-bitbucket * chore: refactoring * added migration script * self review fixes - wip * chore: ProcessIncompleteDeploymentReq updated * updated migration and added util test cases * updated test file name * fix: trigger history audit step * fix: status update for trigger audit * chore: tracer refactoring * removed comment * removed duplicate status updates * added NATS graceful shutdown * fixed wire injection issue * fixed: migration issues * fix: deployment status update * fix: nil UserDeploymentRequest model on save * fix: UserDeploymentRequestRepositoryImpl.FindByCdWfIds nil model * udpated validateStatusUpdate for user deployment request * removed duplicate flag * fix: nil pointer in logger * updated save userDeploymentRequest func * updated save userDeploymentRequest func * udpated: extractAsyncCdDeployRequestFromEventMsg * fix: handled panic issues * updated MarkCurrentDeploymentFailed func * added tracer * fix: unauthorized on async deployment * fixed: updateTriggerEventForIncompleteRequest * fixed envOverride.Environment data * updateTriggerEventForIncompleteRequest for no gitops deployment types * updated CreateHistoriesForDeploymentTrigger * fix: updated history status * fix: create history error * fix: UpdateStatusForCdWfIds * nil pointer check fix * wire gen file * removed unnecessary loggers * fix: logger type * create cm/cs history in transaction * wirer gen file * fix: nil pointer handling * fix: TerminateForPipelineId func * fix: UserDeploymentRequest db struct * updated UpdateCDWorkflowRunnerStatus func * fix: logger noise from markImageScanDeployed * updated auto trigger loggers and ProcessConcurrentAsyncDeploymentReq context * refactored: saveDeploymentTriggerHistory * refactored: trigger async release * fix: GetAllInCompleteRequests query * fix: UNABLE_TO_FETCH timeline handling * fix: async deployment request lock * feat: gitops async with timeline (#5286) * feat: replace user_deployment_request.status with pipeline_status_timelines * updated schema migration script * fix: migration script * update readme to include multi arch flag (#4998) * updated trigger event func and timeline names * review comment incorporation * fix: GetAllInCompleteRequests query * fix: GetAllInCompleteRequests query * fixed: UpdatePreviousQueuedRunnerStatus error * fix: loogers * doc: Added enhancements to security doc (#5203) * Rough Draft Initiated * Created Security Enhancement Docs + Other Adhoc Fixes * Incorporated PM Feedback * Modified Portforward Section to Kubectl Section (#5236) * feat: cd pipeline deployment history refactoring (#5200) * searchableKey service move * migrated cd deployment history code * idempotency of triggerPipeline func and minor fixes * timeline fixes * updated: NewTriggerEvent func * updated ErrorDeploymentSuperseded err handling * updated: GetArgoPipelinesHavingLatestTriggerStuckInNonTerminalStatuses statuses * fix: timeline status * refactored: UpdatePreviousQueuedRunnerStatus * fix: GetPreviousQueuedRunners query * fix: UpdateRunnerStatusToFailedForIds query * fix: buildTriggerEventForOverrideRequest func * fix: UpdateRunnerStatusToFailedForIds query * fix: UpdateRunnerStatusToFailedForIds query * updated: handleAsyncTriggerReleaseError func * fix: concurrent case handlings * updated folder name * updated GetLastStatusPublishedTimeForWfrId func * feat: removed audit timeline and error handlings * fix: FetchTimelinesForWfrIdExcludingStatuses query * fix: deleted pipeline flow * timeline handling * timeline handling fixed * removed unused const * chore: updated go-bitbucket version * removed unnecessary alais --------- Co-authored-by: ayu-devtron <[email protected]> Co-authored-by: ashokdevtron <[email protected]> Co-authored-by: kartik-579 <[email protected]> * main branch merge * vendor files updated * udpated migration number * fix: build issues * chore: updated migration number * fix: context canceling issue * feat: git commit retry * updated migration number * fix: bitbucket git commit issue * feat: env varibales for retry count and delay * migration script updated --------- Co-authored-by: ayu-devtron <[email protected]> Co-authored-by: ashokdevtron <[email protected]> Co-authored-by: kartik-579 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fixes #5255
Checklist:
Does this PR introduce a user-facing change?