Daniel Milroy

Results 47 comments of Daniel Milroy

> Correct, the first resource.acquire response should only contain an up idset. Ok, then I think the resource module is behaving according to the RFC. 1. The two `mark` calls...

My hypothesis is that the production issues were caused by the missing member data in the `resource_pool_t` ctor rather than the resource module. It's possible the JGF reader bug contributed,...

> A parent instance can query its remote child instances to find out when is the earliest by which all the children can allocate requested resources. Then, it should be...

I've confirmed that by manipulating the `at` time in `dfu_traverser_t::run`: https://github.com/flux-framework/flux-sched/blob/35d3c96bb7378ec4f6803a7e340043fd4040e8c1/resource/traversers/dfu.cpp#L277 we can achieve the desired behavior. Here I've simulated this by hardcoding `at = 3600` in `dfu_traverser_t::run` and performing...

> Of course, there will be a decent amount of development required to add new match_op_t cases Actually that is not complicated. > and determine the best way to include...

> I think that t_submit probably makes sense for a default deferred_from value. I'm not quite clear, does the current implementation allow the user to set an absolute time, or...

I started looking into this issue. While it is true that Fluxion features the `rv1_nosched` match format and the `rv1exec` reader, Fluxion [doesn't support recovery with the `rv1_nosched` writer](https://flux-framework.readthedocs.io/projects/flux-sched/en/latest/man5/flux-config-sched-fluxion-resource.html#keys), in...

The intention appears to have been to [use the `rv1` writer for the system instance](https://flux-framework.readthedocs.io/projects/flux-sched/en/latest/man5/flux-config-sched-fluxion-resource.html#example): ```yaml # system-instance will use full-up rv1 writer # so that R will contain scheduling...

> Can you expand on your statement that R_lite does not have the right identifiers to find update existing graph vertices correctly, for those of us that do not know...

I think `{"rank": "-1"}` is just an artifact of running this in `resource-query` vs a running Flux instance.