pipelines-as-code icon indicating copy to clipboard operation
pipelines-as-code copied to clipboard

added support for azure devops

Open aayeshaarshad opened this issue 1 year ago • 9 comments

  • [x] Added support to detect X-Azure-DevOps-EventType in request header
  • [x] Parse payload for push and pull event
  • [x] Created connection with client using pat
  • [x] Get the files inside .tekton directory
  • [x] Ran a successful pipelinerun
  • [x] Set the status; currently tested pending/failed/successful on push
  • [x] Write implementation of missing methods (acl.go, CreateToken, GetFileInsideRepo)
  • [x] Write test cases
  • [x] Do complete e2e testing on both pull and push

aayeshaarshad avatar Apr 08 '24 13:04 aayeshaarshad

/ok-to-test

chmouel avatar Apr 11 '24 12:04 chmouel

Codecov Report

Attention: Patch coverage is 36.24535% with 343 lines in your changes are missing coverage. Please review.

Project coverage is 62.97%. Comparing base (5ecb9ea) to head (a1af7e8). Report is 53 commits behind head on main.

:exclamation: Current head a1af7e8 differs from pull request most recent head fd7c2c3. Consider uploading reports for the commit fd7c2c3 to get more accurate results

Files Patch % Lines
pkg/provider/azuredevops/azuredevops.go 21.91% 279 Missing and 6 partials :warning:
pkg/provider/azuredevops/acl.go 0.00% 19 Missing :warning:
pkg/provider/azuredevops/parse_payload.go 85.14% 10 Missing and 5 partials :warning:
pkg/reconciler/event.go 0.00% 11 Missing and 1 partial :warning:
pkg/kubeinteraction/labels.go 25.00% 4 Missing and 2 partials :warning:
pkg/adapter/adapter.go 40.00% 2 Missing and 1 partial :warning:
pkg/provider/azuredevops/detect.go 88.88% 2 Missing and 1 partial :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1654      +/-   ##
==========================================
- Coverage   64.31%   62.97%   -1.34%     
==========================================
  Files         140      144       +4     
  Lines       10789    11326     +537     
==========================================
+ Hits         6939     7133     +194     
- Misses       3334     3661     +327     
- Partials      516      532      +16     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Apr 11 '24 12:04 codecov[bot]

/test go-testing

chmouel avatar Apr 16 '24 12:04 chmouel

@chmouel any update on review?

aayeshaarshad avatar May 15 '24 09:05 aayeshaarshad