Skip to content

feat: add pipeline step for bitbucket#4459

Merged
warren830 merged 2 commits intoapache:mainfrom
merico-ai:issue/4340-2
Feb 20, 2023
Merged

feat: add pipeline step for bitbucket#4459
warren830 merged 2 commits intoapache:mainfrom
merico-ai:issue/4340-2

Conversation

@likyh
Copy link
Contributor

@likyh likyh commented Feb 20, 2023

Summary

  1. collect pipeline step as cicd task
  2. add page,pagelen,size to fix a bug(to collect all page).

Does this close any open issues?

Closes #4340

Screenshots

Result like this:

bitbucket:BitbucketPipelineStep:1:{257bf6fd-4411-472b-82cf-0291d73d53ed},2023-02-20 12:09:33.981830 +00:00,2023-02-20 12:09:33.981830 +00:00,"{""ConnectionId"":1,""FullName"":""likyh/likyhphp""}",_raw_bitbucket_pipeline_steps,11670,"",staging,{990e12db-7802-4612-9713-b998dc16f468},SUCCESS,DONE,DEPLOYMENT,STAGING,8,2023-02-20 09:30:08.751166 +00:00,2023-02-20 09:30:16.433231 +00:00,""

Type string `gorm:"type:varchar(255)"`
Name string `gorm:"type:varchar(255)"`
Environment string `gorm:"type:varchar(255)"`
EnvironmentType string `gorm:"type:varchar(255)"`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

add StepId / Environment / EnvironmentType

Generate(data.Options.ConnectionId, bitbucketDeployment.Name),
PipelineId: bitbucketDeployment.PipelineId,
Name: bitbucketPipelineStep.Name,
PipelineId: bitbucketPipelineStep.PipelineId,
Copy link
Contributor

Choose a reason for hiding this comment

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

we should use domain if of cicd pipeline

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image
fixed

Copy link
Contributor

@warren830 warren830 left a comment

Choose a reason for hiding this comment

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

Only a minor issue

Copy link
Contributor

@warren830 warren830 left a comment

Choose a reason for hiding this comment

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

A minor issue need to be fixed

Copy link
Contributor

@warren830 warren830 left a comment

Choose a reason for hiding this comment

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

LGTM

@warren830 warren830 merged commit b1d4a2c into apache:main Feb 20, 2023
@warren830 warren830 deleted the issue/4340-2 branch February 20, 2023 16:29
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.

[Epic][bitbucket] some new feature transfer to bitbucket

2 participants