Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: added reactive planning package #40

Merged
merged 2 commits into from
Aug 9, 2021
Merged

feat: added reactive planning package #40

merged 2 commits into from
Aug 9, 2021

Conversation

richardcase
Copy link
Member

@richardcase richardcase commented Jul 29, 2021

What this PR does / why we need it:
A package to perform reactive planning has been added. This will be used to make a plan of the actions that need to be done to perform an operation for a microvm.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

Checklist:

  • squashed commits
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

Release note:

Reactive planning framework added.

@richardcase richardcase added the kind/feature New feature or request label Jul 29, 2021
bigkevmcd
bigkevmcd previously approved these changes Aug 2, 2021
Copy link

@bigkevmcd bigkevmcd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

pkg/id/id.go Show resolved Hide resolved
A package to perform reactive planning has been added. This will be
used to make a plan of the actions that ned to be done to perform
an operation for a microvm.

Signed-off-by: Richard Case <[email protected]>
Slight change to the id package to add a default random
source.

Signed-off-by: Richard Case <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature or request size/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants