Skip to content
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.

Transition to using python-bugzilla library #10

Merged
merged 1 commit into from Jan 15, 2019

Conversation

smunilla
Copy link
Contributor

@smunilla smunilla commented Jan 10, 2019

Move away from using shell callouts to the python-bugzilla cli and use the actual library. Also remove the deprecated bug transition searches now that we can add VERIFIED bugs regardless of the path they took to get there.

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 10, 2019
@openshift-ci-robot openshift-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jan 10, 2019
@smunilla smunilla changed the title [WIP] Transition to using python-bugzilla library Transition to using python-bugzilla library Jan 11, 2019
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 11, 2019
else:
bug_ids = [elliottlib.bugzilla.Bug(id=i) for i in id]
bzapi = elliottlib.bzutil.get_bzapi(bz_data)
bug_ids = [bzapi.getbug(i) for i in id]
Copy link
Contributor

Choose a reason for hiding this comment

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

We are the best at duplicating code

Copy link
Contributor

@sosiouxme sosiouxme left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@smunilla smunilla merged commit 055f812 into openshift-eng:master Jan 15, 2019
@smunilla smunilla deleted the convert-to-bzlib branch January 15, 2019 19:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants