Bug 602215 - Reservation emails should be more Beaker-friendly
Summary: Reservation emails should be more Beaker-friendly
Alias: None
Product: Beaker
Classification: Community
Component: tests   
(Show other bugs)
Version: 0.5
Hardware: All
OS: Linux
medium vote
Target Milestone: ---
Assignee: Bill Peck
QA Contact:
: 603757 (view as bug list)
Depends On:
TreeView+ depends on / blocked
Reported: 2010-06-09 12:18 UTC by David Kovalsky
Modified: 2014-03-31 23:45 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2012-08-14 00:14:14 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

Comment 1 Marian Csontos 2010-06-09 12:43:07 UTC
This is to be filled in against reservesys test as it is the reservesys which needs to be adapted a bit, but I do not mind making the changes...

RESULT_SERVER is a proxy running on a lab machine (or system as they are called now)

There is one problem: LM's have no access to Scheduler (actually, they do not know about it.)

This is the only thing which should be raised against beaker. Bill, could you export it in job XML please?

Comment 2 Marian Csontos 2010-06-11 13:00:54 UTC
Took this as a relax item ;-)

Re: changing From/Subject: though it makes sense, if anyone uses this in a script, it will break them.

Comment 3 David Kovalsky 2010-06-11 13:16:46 UTC
I think it's pretty hard to do filtering by the subject if you only have a redhat.com FQDN :) But agreed, the possibility is there. 

For making filtering easier, you could also add more information as email headers. for example, X-BEAKER-JOBID: 2418, X-BEAKER-ACTION: reservsys, etc...

With one value per header is should be way easier to setup automatic filtering.

Comment 4 Marian Csontos 2010-06-11 14:06:32 UTC
FYI, there is already X-RHTS-test header

But I agree.

I am going on...

Comment 5 Kevin Baker 2010-06-11 14:59:35 UTC
(In reply to comment #4)
> FYI, there is already X-RHTS-test header

We should standardies the naming. I'm asking the guys to s/rhts/beaker/ elsewhere so these should be consistent too.

X-Beaker-<key> <value>

Comment 6 Marian Csontos 2010-06-14 14:28:13 UTC
*** Bug 603757 has been marked as a duplicate of this bug. ***

Comment 7 Marian Csontos 2010-06-17 19:02:50 UTC
Fixed by rh-tests-distribution-reservesys-2.0-44

I need variable with scheduler's URL. Could you sent it out in XML?

Assign back to me afterwards please

Comment 8 Dan Callaghan 2012-08-14 00:14:14 UTC
All the suggestions in this bug have made it into the latest version of Beaker and /distribution/reservesys.

Note You need to log in before you can comment on or make changes to this bug.