Skip to content

feat: grow/shrink support for fluxion#12

Open
vsoch wants to merge 3 commits intomainfrom
add-grow-support
Open

feat: grow/shrink support for fluxion#12
vsoch wants to merge 3 commits intomainfrom
add-grow-support

Conversation

@vsoch
Copy link
Copy Markdown
Member

@vsoch vsoch commented Oct 31, 2024

This changeset exposes the (WIP) API to allow a resource graph to grow, which needs to be a call to the c api "grow" that gets passed to unpack_at. This is a WIP because the container build has a custom clone/build of flux-sched with the grow-api branch, and we will need to eventually update that.

Note that the test isn't currently working (and will fail) but I'm opening the PR to see the changed files easily.

@vsoch vsoch force-pushed the add-grow-support branch 2 times, most recently from e4c421b to 78bec86 Compare November 1, 2024 06:06
This changeset exposes the (WIP) API to allow a resource
graph to grow, which needs to be a call to the c api
"grow" that gets passed to unpack_at. This is a WIP
because the container build has a custom clone/build of
flux-sched with the grow-api branch, and we will need
to eventually update that.

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
@vsoch
Copy link
Copy Markdown
Member Author

vsoch commented Nov 1, 2024

Note that the tests can't pass here because they aren't using the PR branch. They pass locally.

This currently uses a custom branch, and will need to be
updated when merged into flux-sched. This last change
updates the CI to use the latest noble image, and
adds back the shrink support and test.

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
I tested having multiple contexts, but cancel issued
an error and I did not think now was worth trying to
debug.

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
@vsoch vsoch changed the title feat: grow support for fluxion feat: grow/shrink support for fluxion Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant