-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Add Migrations for Enclave Changes to UserCode and CodeHistory #8986
Add Migrations for Enclave Changes to UserCode and CodeHistory #8986
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, good stuff Brendan!
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
f733a03
to
242c4a0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM !! Made changes to UserCode.
6d75878
to
242c4a0
Compare
Description
Add migrations for UserCode and CodeHistory for enclave changes. Includes upgrades to UserCode and SubmitUserCode to add deployment policies. Will have to be updated based on naming changes to deployment policies in #8981 so PR is WIP for now.
Closes
Affected Dependencies
List any dependencies that are required for this change.
How has this been tested?
obj.migrate_to(version_no)
to ensure no errors.Checklist