Closed
Bug 936603
Opened 11 years ago
Closed 10 years ago
in the slave failure page on Ouija, determine the timestamp related to the last job ran and subtract that from the current time
Categories
(Testing :: General, defect)
Testing
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: jmaher, Unassigned)
References
Details
Add a "time of last job" column (in relative times if that's ok). We
could populate this dynamically client side, similar to the slave health
page.
I am not sure if we have this exact information, but I suspect we do and can make a new column on the table with this item.
Reporter | ||
Comment 2•11 years ago
|
||
oh, this is missing :(
We have starttime and endtime to calculate duration:
https://github.com/dminor/ouija/blob/master/src/updatedb.py#L122
we also store the date, but this is the date of the push to the tree.
Should we change what date we store?
Should we add another field for job start or job end?
:edmorley- what would be a useful time to base off, starttime or endtime? I would vote for starttime
:dminor- would you prefer a new field for the timestamp or changing the date that we use? I would vote for changing the date
Flags: needinfo?(emorley)
Flags: needinfo?(dminor)
Comment 3•11 years ago
|
||
(In reply to akruglov from comment #1)
> How this data can be fetched?
> Is there some API?
See bug 936610 comment 2 :-)
(In reply to Joel Maher (:jmaher) from comment #2)
> :edmorley- what would be a useful time to base off, starttime or endtime? I
> would vote for starttime
Yeah starttime :-)
Flags: needinfo?(emorley)
(In reply to Ed Morley [:edmorley UTC+1] from comment #3)
> (In reply to akruglov from comment #1)
> > How this data can be fetched?
> > Is there some API?
>
> See bug 936610 comment 2 :-)
>
It requires authentication.
Comment 6•11 years ago
|
||
(In reply to akruglov from comment #5)
> It requires authentication.
Yeah this is expected (and unavoidable afaik) to set this up test, you may need to request that your LDAP account have the appropriate bits set to access slavealloc (see bug 887033 for an example of such a request).
Callek - is there any way we could get read-only access to slavealloc without requiring non-default LDAP bits be set?
Flags: needinfo?(bugspam.Callek)
Comment 7•11 years ago
|
||
(In reply to Ed Morley [:edmorley UTC+1] from comment #6)
> (In reply to akruglov from comment #5)
> > It requires authentication.
>
> Yeah this is expected (and unavoidable afaik) to set this up test, you may
> need to request that your LDAP account have the appropriate bits set to
> access slavealloc (see bug 887033 for an example of such a request).
>
> Callek - is there any way we could get read-only access to slavealloc
> without requiring non-default LDAP bits be set?
I don't think so, but I'm not an expert here, redirecting to :dustin
Flags: needinfo?(bugspam.Callek)
Updated•11 years ago
|
Flags: needinfo?(dustin)
Comment 8•11 years ago
|
||
Slavealloc doesn't distinguish read-only from read-write, so that's not possible. We're simultaneously starting to make the data in slavealloc more security-sensitive, so having external tools talking to it is probably not a good idea.
Flags: needinfo?(dustin)
Reporter | ||
Comment 9•11 years ago
|
||
can we make the process/tool when enabling/disabling a slave write to a secondary data source that is public?
Comment 10•11 years ago
|
||
(Even if it's just enable/disable state)
Comment 11•11 years ago
|
||
I think that would be slaveapi. In fact, I'd argue we should make slavealloc a component of slaveapi.
Comment 12•10 years ago
|
||
We're planning to move this functionality to Treeherder anyway (Bug 1071152) so there is no point in fixing it here.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•