testsuite: add test for expiration update inheritance

Problem: No tests in the testsuite ensure that an expiration update
of a job containing a Flux instance is inherited by the subinstance
Fluxion scheduler.

Add a test to t4011-match-duration.t that launches a subinstance,
extends the duration of that instance, then ensures that new jobs
submitted to the instance inherit the updated expiration.

Author: grondo

t/t4011-match-duration.t

@@ -7,8 +7,9 @@ test_description='Test that parent duration is inherited according to RFC14'
 #
 # test_under_flux is under sharness.d/
 #
-test_under_flux 1
+test_under_flux 2
 
+flux setattr log-stderr-level 1
 export FLUX_URI_RESOLVE_LOCAL=t
 
 # Ensure fluxion modules are loaded under flux-alloc(1)
@@ -34,6 +35,35 @@ test_expect_success 'load fluxion modules in parent instance' '
 	load_qmanager &&
 	test_debug "flux dmesg -H | grep version"
 '
+# Usage: job_manager_get_R ID
+# This needs to be a shell script since it will be run under flux-proxy(1):
+cat <<'EOF' >job_manager_get_R
+#!/bin/sh
+flux python -c \
+"import flux; \
+ payload = {\"id\":$(flux job id $1),\"attrs\":[\"R\"]}; \
+ print(flux.Flux().rpc(\"job-manager.getattr\", payload).get_str()) \
+"
+EOF
+chmod +x job_manager_get_R
+
+export PATH=$(pwd):$PATH
+
+subinstance_get_R() {
+	flux proxy $1 flux kvs get resource.R
+}
+subinstance_get_expiration() {
+	subinstance_get_R $1 | jq .execution.expiration
+}
+# Usage: subinstance_get_duration ID JOBID
+subinstance_get_job_duration() {
+	flux proxy $1 job_manager_get_R $2 |
+		jq '.R.execution | .expiration - .starttime'
+}
+subinstance_get_job_expiration() {
+	flux proxy $1 job_manager_get_R $2 | jq '.R.execution.expiration'
+}
+
 test_expect_success HAVE_JQ 'parent expiration is inherited when duration=0' '
 	cat >get_R.sh <<-EOT &&
 	#!/bin/sh
@@ -59,8 +89,38 @@ test_expect_success HAVE_JQ 'parent expiration is inherited when duration=0' '
 	echo $exp2 | jq ". == $expiration" &&
 	flux shutdown --quiet $jobid
 '
+#  Check if running job updates are supported:
+id=$(flux submit sleep inf)
+flux update $id duration=1m && test_set_prereq FLUX_UPDATE_RUNNING
+flux cancel $id
+
+test_expect_success FLUX_UPDATE_RUNNING \
+	'expiration update is detected by subinstance scheduler' '
+	id=$(flux alloc --bg -t5m -n2) &&
+	exp1=$(subinstance_get_expiration $id) &&
+	test_debug "echo instance expiration is $exp1" &&
+	id1=$(flux proxy $id flux submit sleep 300) &&
+	duration1=$(subinstance_get_job_duration $id $id1) &&
+	test_debug "echo initial duration of subinstance job1 is $duration1" &&
+	echo $duration1 | jq -e ". < 300" &&
+	test_debug "echo updating duration of alloc job +5m" &&
+	flux update $id duration=+5m &&
+	test_debug "echo waiting for resource-update event" &&
+	flux proxy $id flux kvs eventlog wait-event -vt 30 \
+		resource.eventlog resource-update &&
+	exp2=$(subinstance_get_expiration $id) &&
+	test_debug "echo expiration updated from $exp1 to $exp2" &&
+	echo $exp2 | jq -e ". == $exp1 + 300" &&
+	id2=$(flux proxy $id flux submit sleep 300) &&
+	duration2=$(subinstance_get_job_duration $id $id2) &&
+	test_debug "echo duration of subinstance job2 is $duration2" &&
+	echo $duration2 | jq -e ". > 300" &&
+	flux proxy $id flux cancel --all &&
+	flux shutdown --quiet $id
+'
 test_expect_success 'unload fluxion modules' '
 	remove_qmanager &&
-	remove_resource
+	remove_resource &&
+	flux module load sched-simple
 '
 test_done