Parameterize bzcache server

RESOLVED FIXED

Status

RESOLVED FIXED
7 years ago
4 years ago

People

(Reporter: mcote, Unassigned)

Tracking

Details

Attachments

(4 attachments)

(Reporter)

Description

7 years ago
The bzcache module uses a hardcoded ES location.  This should be a parameter.
(Reporter)

Comment 1

7 years ago
Created attachment 587406 [details] [diff] [review]
Make ES server address a parameter

This makes the ES server address an optional parameter.
Attachment #587406 - Flags: review?(jgriffin)
Comment on attachment 587406 [details] [diff] [review]
Make ES server address a parameter

thanks!
Attachment #587406 - Flags: review?(jgriffin) → review+
(Reporter)

Comment 3

7 years ago
Created attachment 587409 [details] [diff] [review]
Use same ES server as rest of ES calls for bzcache.

This just updates the orangefactor server to use the same ES server for
bzcache as is used elsewhere.  I also threw in a tiny fix for the default
start date.
Attachment #587409 - Flags: review?(jgriffin)
(Reporter)

Updated

7 years ago
Attachment #587406 - Attachment is obsolete: true
Attachment #587409 - Flags: review?(jgriffin) → review+
(Reporter)

Comment 4

7 years ago
Created attachment 587432 [details] [diff] [review]
Make BZAPI server address a parameter

Might as well parameterize the BZ API server while we're at it...
Attachment #587432 - Flags: review?(jgriffin)
(Reporter)

Comment 5

7 years ago
Created attachment 587434 [details] [diff] [review]
Use same BZ API server for bzcache

Pass the BZAPI server into bzcache.  I also corrected the ES server to the
proper buildbot server.
Attachment #587434 - Flags: review?(jgriffin)
(Reporter)

Updated

7 years ago
Attachment #587432 - Attachment is obsolete: true
Attachment #587432 - Flags: review?(jgriffin)
(Reporter)

Comment 6

7 years ago
Comment on attachment 587406 [details] [diff] [review]
Make ES server address a parameter

Oops didn't mean to obsolete this.
Attachment #587406 - Attachment is obsolete: false
(Reporter)

Comment 7

7 years ago
Comment on attachment 587432 [details] [diff] [review]
Make BZAPI server address a parameter

Nor this.
Attachment #587432 - Attachment is obsolete: false
Attachment #587432 - Flags: review+
Attachment #587434 - Flags: review?(jgriffin) → review+
(Assignee)

Updated

4 years ago
Product: Testing → Tree Management
You need to log in before you can comment on or make changes to this bug.