Skip to content

fix(azuredevops_go): persist build run title and url#8933

Open
bujjibabukatta wants to merge 1 commit into
apache:mainfrom
bujjibabukatta:fix/#8887
Open

fix(azuredevops_go): persist build run title and url#8933
bujjibabukatta wants to merge 1 commit into
apache:mainfrom
bujjibabukatta:fix/#8887

Conversation

@bujjibabukatta

Copy link
Copy Markdown
Contributor

Fixes #8887.

This persists Azure DevOps buildNumber and build URL in the azuredevops_go build tool table, then propagates them to cicd_pipelines and cicd_pipeline_commits as display_title/url. Regex enrichment for pipeline type/environment now also considers buildNumber, so deployment/prod identifiers present in Azure run names are detected even when definition.name only contains the pipeline name.

Also adds a migration for existing installs and updates E2E fixtures/snapshots to cover buildNumber-based deploy/prod matching.

AzuredevopsId int `gorm:"primaryKey"`
RepositoryId string `gorm:"type:varchar(255)"`
Name string `gorm:"type:varchar(100)"`
BuildNumber string `gorm:"type:varchar(255)"`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please do NOT change historical script! this is utterly wrong.

type addBuildRunFields struct{}

type BuildRunFields struct {
ConnectionId uint64 `gorm:"primaryKey"`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please omit unchanged columns.

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.

Azure DevOps (azuredevops_go): buildNumber not stored/exposed → empty display_title / scope regex misses deploy segment

2 participants