Synchronise master with upstream - #120
Closed
github-actions[bot] wants to merge 14 commits into
Closed
Conversation
With the flavor reservation plugin, it has made some calcuation on what the flavor chosen means in terms of resources. This is exactly the sort of information that is needed by an external enfrocement service, to track how many resource hours are being comsumted by each reservation. As such, we find a way to send this extra information to the external enforcement service. Change-Id: I168a2e90735226676cd573fc1a3c316efde1f6d1
This change allows for users to configure blazar to call out to an external enforcement API with a bearer token. This is very handy when you create an enforcement API using the Django Rest Framework. Change-Id: I772db7c85620d7a9f0fc1b7aa10f73a96bc86a5f
To keep track of resource usage in an external enforcement service, its really important for the lease_id to be passed to the external service. Change-Id: I953cf73891f27bd649ba27db9769f11f8fe98d59
The follow on change was cusing problems with the tox complexity checker. This gives that change some head room. Change-Id: Ic2378b6b7e856f99b31fcb2c8eb1dd1e0742b05d
If we use check_create to keep state about reserverations we need an on_end callback to be attempted when the lease is deleted early due to a failure, such as reserve_resource failing due to some race with another reservation request. In other error cases, we would expect a lease db object to still be created at the end of the create lease call, such that delete will be called that triggers on_end. One alternative is to move check_create after operations such as reserve_resource, but in that case we need to make all of the plugins capable of cleaning up the resources they have just created, which seems a bad idea when it is not uncommon that a request is rejected due to the enforcement logic. Change-Id: If4ff1c8da32c3be47bf1523058f83effd7340f02
Option to just do the enforcement check and resource allocations check, but not actually create the resources. Change-Id: I150e3f0caad1ab5196d54e75d0ff366ec66797a7
C901 Function is too complex is catching ManagerService.create_lease() because the function has more than 5 potential branches. Because the function is in WIP in master branch, ignore C901 at least until the function is in stable state.
Add C901 to ignore list of pep8
feat: automatic update of workflows stackhpc/master
feat: automatic update of community files stackhpc/master
github-actions
Bot
added
automated
Alex-Welsh
force-pushed
the
stackhpc/master
branch
from
July 30, 2026 09:53
41bb69b to
ae64257
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains a snapshot of master from upstream master.