Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Look before you leap. #5

Merged
merged 2 commits into from Nov 19, 2015
Merged

Look before you leap. #5

merged 2 commits into from Nov 19, 2015

Conversation

ralphbean
Copy link
Contributor

It used to be that bugzilla's messaging plugin sent only a timestamp and
the bug_id of the bug that changed. We had to call back to RHBZ over the
xmlrpc API right off the bat in order to figure out if this was a Fedora or
Fedora EPEL bug that we cared about.

As of this ticket,
we are now able to check the product supplied in the original message, so
that we can reduce the strain we put on RHBZ's xmlrpc api.

It used to be that bugzilla's messaging plugin sent only a timestamp and the
bug_id of the bug that changed.  We had to call back to RHBZ over the xmlrpc
API right off the bat in order to figure out if this was a Fedora or Fedora
EPEL bug that we cared about.

As of [this ticket](https://bugzilla.redhat.com/show_bug.cgi?id=1248259), we
are now able to check the product supplied in the original message, so that we
can reduce the strain we put on RHBZ's xmlrpc api.
@pypingou
Copy link
Member

Cool 👍

ralphbean added a commit that referenced this pull request Nov 19, 2015
@ralphbean ralphbean merged commit 67060c0 into develop Nov 19, 2015
@ralphbean ralphbean deleted the feature/reduce-bz-load branch November 19, 2015 16:10
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.

None yet

2 participants