BryanDavis has uploaded a new change for review.
https://gerrit.wikimedia.org/r/287999
Change subject: www: Another round of undef index fixes
......................................................................
www: Another round of undef index fixes
More hack fixes to avoid undef index warnings. The whole status page
probably needs to be rethought at some point but this should at least
make the PHP interpreter a bit happier with us.
Change-Id: Icc53b3ecb6de7d1bd13239998fae79e75ccee630
---
M www/content/status.php
1 file changed, 13 insertions(+), 12 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/labs/toollabs
refs/changes/99/287999/1
diff --git a/www/content/status.php b/www/content/status.php
index 28c5bf8..6ff1e40 100644
--- a/www/content/status.php
+++ b/www/content/status.php
@@ -98,24 +98,25 @@
foreach ( $xjob->jobvalue as $jv ) {
$job[(string) $jv->attributes()->name] =
(string) $jv;
}
- $jobs[$jid]['state'] = $job['job_state'];
- if ( stristr( $job['job_state'], 'R' ) !== false ) {
+ $rawState = array_get( $job, 'job_state' );
+ $jobs[$jid]['state'] = $rawState;
+ if ( stristr( $rawState, 'R' ) !== false ) {
$jobs[$jid]['state'] = 'Running';
}
- if ( stristr( $job['job_state'], 's' ) !== false ) {
+ if ( stristr( $rawState, 's' ) !== false ) {
$jobs[$jid]['state'] = 'Suspended';
}
- if ( stristr( $job['job_state'], 'd' ) !== false ) {
+ if ( stristr( $rawState, 'd' ) !== false ) {
$jobs[$jid]['state'] = 'Deleting';
}
$jobs[$jid]['host'] = $hname;
- $jobs[$jid]['priority'] = $job['priority'];
+ $jobs[$jid]['priority'] = array_get( $job, 'priority' );
$host['jobs'][] = $jid;
}
foreach ( $xhost->hostvalue as $hv ) {
$host[(string) $hv->attributes()->name] = (string) $hv;
}
- $host['mem'] = mmem( $host['mem_used'] ) / mmem(
$host['mem_total'] );
+ $host['mem'] = mmem( array_get( $host, 'mem_used', '0M' ) ) /
mmem( array_get( $host, 'mem_total', '1M' ) );
$hosts[$hname] = $host;
}
}
@@ -129,9 +130,9 @@
ksort( $jobs );
foreach ( $hosts as $host => $h ) {
- $hvmem = $h['h_vmem'];
+ $hvmem = array_get( $h, 'h_vmem', 0 );
foreach ( $h['jobs'] as $jn ) {
- $hvmem -= $jobs[$jn]['h_vmem'];
+ $hvmem -= array_get( $jobs[$jn], 'h_vmem', 0 );
}
$hvmem = (int) ( $hvmem / 1024 / 1024 );
if ( $hvmem < 0 ) {
@@ -140,9 +141,9 @@
?>
<div class="hostline">
<span class="hostname"><?= htmlspecialchars( $host ) ?></span>
-<b>Load:</b> <?= (int) ( $h['load_avg'] * 1000 ) / ( $h['num_proc'] * 10 ) ?>%
-<b>Memory:</b> <?= (int) ( $h['mem'] * 1000 ) / 10 ?>%
-<?php if ( $h['h_vmem'] > 0 ) { ?>
+<b>Load:</b> <?= (int) ( array_get( $h, 'load_avg', 0 ) * 1000 ) / (
array_get( $h, 'num_proc', 1 ) * 10 ) ?>%
+<b>Memory:</b> <?= (int) ( array_get( $h, 'mem', 0 ) * 1000 ) / 10 ?>%
+<?php if ( array_get( $h, 'h_vmem', 0 ) > 0 ) { ?>
<b>Free vmem:</b> <?= humanmem( $hvmem ); ?>
<?php } ?>
</div>
@@ -176,7 +177,7 @@
<?= array_key_exists('vmem', $j) ? sprintf( '%d/%d', humanmem( $j['vmem'] /
1024 / 1024 ), humanmem( array_get( $j, 'h_vmem', 0 ) / 1024 / 1024 ) ) : 'n/a'
?>
<?php
if ( array_key_exists('maxvmem', $j) &&
- $j['maxvmem'] > $j['vmem'] * 1.02
+ $j['maxvmem'] > array_get( $j, 'vmem', 0 ) * 1.02
) {
?>
(peak <?= humanmem( $j['maxvmem'] / 1024 / 1024 ) ?>)
--
To view, visit https://gerrit.wikimedia.org/r/287999
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icc53b3ecb6de7d1bd13239998fae79e75ccee630
Gerrit-PatchSet: 1
Gerrit-Project: labs/toollabs
Gerrit-Branch: master
Gerrit-Owner: BryanDavis <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
