fix(STONEINTG-709): add support missing info of build PLR error

[kasemAlem]

Fix STONEINTG-709 , adding new error handler for the missing params error returned by build PipelineRun and stop the PLR to be returned to the reconcile queue .

• Commit messages are descriptive enough (hints)
• Code coverage from testing does not decrease and new code is covered (check the PR coverage on codecov)
• Controllers diagrams are updated when PR changes controllers code (if applicable)

[hongweiliu17]

We may also replace the errors such as fmt.Errorf("couldn't find the CHAINS-GIT_URL PipelineRun result") link with MissingInfoInPipelineRunError() and check the error returned by prepareSnapshotForPipelineRun(), and then update the build PLR annotation and ContinueProcessing() if IsMissingInfoInPipelineRunError(err) is true

[kasemAlem]

#506 (comment)
@hongweiliu17 is there a specific build PLR annotation that I should update or I should create a new one ?

[hongweiliu17]

#506 (comment) @hongweiliu17 is there a specific build PLR annotation that I should update or I should create a new one ?

You may use this one https://github.com/redhat-appstudio/integration-service/pull/489/files#diff-16fb634f123aeba9508751c94b6a61320bf6877bdb01ff6f00da2a72b92ee432R3

[codecov]

Codecov Report

Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (90837ac) 69.14% compared to head (dbf37fd) 69.35%.
Report is 1 commits behind head on main.

Files	Patch %	Lines
controllers/buildpipeline/buildpipeline_adapter.go	0.00%	8 Missing ⚠️
tekton/utils.go	87.50%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #506      +/-   ##
==========================================
+ Coverage   69.14%   69.35%   +0.21%     
==========================================
  Files          44       44              
  Lines        4995     5013      +18     
==========================================
+ Hits         3454     3477      +23     
+ Misses       1208     1205       -3     
+ Partials      333      331       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kasemAlem]

#506 (comment) @hongweiliu17 is there a specific build PLR annotation that I should update or I should create a new one ?

You may use this one https://github.com/redhat-appstudio/integration-service/pull/489/files#diff-16fb634f123aeba9508751c94b6a61320bf6877bdb01ff6f00da2a72b92ee432R3

thanks @hongweiliu17 , I've used the same one here

[dirgim]

LGTM

[kasemAlem]

/test integration-service-e2e

[openshift-ci]

New changes are detected. LGTM label has been removed.

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud