Skip to content

Added support for pipeline bridges#1203

Merged
max-wittig merged 2 commits intopython-gitlab:masterfrom
intostern:feat/bridge
Oct 8, 2020
Merged

Added support for pipeline bridges#1203
max-wittig merged 2 commits intopython-gitlab:masterfrom
intostern:feat/bridge

Conversation

@intostern
Copy link
Copy Markdown

@intostern intostern commented Oct 2, 2020 •
edited
Loading

Hi,

If you use child pipelines, in your project, i.e.:

security_scans:
  stage: security_scanning
  only:
    - master
  trigger:
    include:
      - local: ".gitlab-ci/security.yml"

,

then API call pipeline.jobs.list() doesn't include child pipelines into output.
There is a different API call for this: https://docs.gitlab.com/ee/api/jobs.html#list-pipeline-bridges

With this PR you'll be able to exeute:
bridges = pipeline.bridges.list()

nejch
nejch requested changes Oct 2, 2020
View reviewed changes
Copy link
Copy Markdown
Member

@nejch nejch left a comment

Choose a reason for hiding this comment

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

Thanks a lot for this addition! :) I just added a few comments.

Would you mind adding some unit tests for this? You can find examples here: https://github.com/python-gitlab/python-gitlab/tree/master/gitlab/tests/objects

Also, to make the build green you will need to amend your commit message to comply with conventional commits, see instructions here: https://github.com/python-gitlab/python-gitlab/#development-workflow

Let me know if you get stuck.

@intostern intostern closed this Oct 7, 2020
@intostern
Copy link
Copy Markdown
Author

intostern commented Oct 7, 2020

Need to re-create commit messages

@intostern intostern deleted the feat/bridge branch October 7, 2020 13:37
@nejch
Copy link
Copy Markdown
Member

nejch commented Oct 7, 2020

Need to re-create commit messages

@intostern You can force-push to the same branch after fixing/squashing, no need to close the pull request 😉 git push -f

@intostern intostern restored the feat/bridge branch October 7, 2020 16:37
@intostern intostern reopened this Oct 7, 2020
@intostern
Copy link
Copy Markdown
Author

intostern commented Oct 7, 2020

Need to re-create commit messages

@intostern You can force-push to the same branch after fixing/squashing, no need to close the pull request 😉 git push -f

Thanks, will force-push

@intostern intostern force-pushed the feat/bridge branch 2 times, most recently from 8cdce6f to 2cdefe0 Compare October 7, 2020 17:27
@intostern intostern closed this Oct 7, 2020
@intostern intostern reopened this Oct 7, 2020
@codecov-io
Copy link
Copy Markdown

codecov-io commented Oct 7, 2020

Codecov Report

Merging #1203 into master will increase coverage by 0.05%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1203      +/-   ##
==========================================
+ Coverage   77.50%   77.56%   +0.05%     
==========================================
  Files          11       11              
  Lines        2819     2826       +7     
==========================================
+ Hits         2185     2192       +7     
  Misses        634      634
Impacted Files Coverage Δ
gitlab/v4/objects/__init__.py 79.12% <100.00%> (+0.07%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 40ec2f5...05cbdc2. Read the comment docs.

@intostern
Copy link
Copy Markdown
Author

intostern commented Oct 8, 2020

Hi @nejch ,

All done.

@max-wittig
Copy link
Copy Markdown
Member

max-wittig commented Oct 8, 2020

Thanks for the MR. Looks good!

@max-wittig max-wittig merged commit c303dab into python-gitlab:master Oct 8, 2020
@zakhenry
Copy link
Copy Markdown

zakhenry commented Jan 27, 2021

Hi I'd like to use this feature but it seems like it was merged since the latest release of this library. Is there a way to install a prerelease/nightly version so I can access this feature. Also is there any eta on this feature being released?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nejch nejch nejch requested changes

Assignees

No one assigned

Labels

hacktoberfest-accepted

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@intostern @nejch @codecov-io @max-wittig @zakhenry