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.

Sign up for GitHub

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

Jump to bottom

Data migration + testing #8854

Open
wants to merge 58 commits into
base: dev
Choose a base branch
from
Open

Data migration + testing #8854

wants to merge 58 commits into from

Conversation

koenvanderveen
Copy link
Collaborator

@koenvanderveen koenvanderveen commented May 24, 2024 •
edited by eelcovdw
Loading

Description

TODO

  • wait_for.sh check in k8s migration tox command throws error.
  • add tox migration.test and migration.k8s.test to CI. New action or add to existing action?

Affected Dependencies

List any dependencies that are required for this change.

How has this been tested?

  • Describe the tests that you ran to verify your changes.
  • Provide instructions so we can reproduce.
  • List any relevant details for your test configuration.

Checklist

@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@madhavajay madhavajay added this to the 24Q3 - G1M3 milestone Jul 1, 2024
eelcovdw and others added 20 commits July 3, 2024 14:17
@eelcovdw
deployment for migration
3562d0d
@eelcovdw
finalize migration for local setup
e1d8c91
@eelcovdw
Merge pull request #9006 from OpenMined/eelco/migrate-to-new-node
87eb464 
migrate to new node
@eelcovdw
remove debug flag
0479e18
@eelcovdw
merge dev
5c99c69
@eelcovdw
disable migration by default
504498b
@eelcovdw
Merge branch 'dev' into migration-tests
7964586
@eelcovdw
serde + permissions bugfixes
58e8dc4
@eelcovdw
migration fixes
c1b5c2a
@eelcovdw
update notebook
aa103b8
@eelcovdw
merge dev
2455d0b
@eelcovdw
min blob storage = 0
301cbd4
@eelcovdw
disable blob/local storage tests
f4fc883
@eelcovdw
Revert "disable blob/local storage tests"
e8b45af 
This reverts commit f4fc883.
@eelcovdw
Revert "min blob storage = 0"
18b9a21 
This reverts commit 301cbd4.
@eelcovdw
add unittests
8154715
@eelcovdw
comments
efd1030
@khoaguin
[syft/blob_storage] only add blob persmission for an action object wh…
54ba6b3 
…en its `syft_blob_storage_entry_id` is not None
@khoaguin
[syft/action_service] only add blob permission if the entry's id is n…
787af9b 
…ot None
@khoaguin
Merge branch 'dev' into migration-tests
a349876
@khoaguin khoaguin mentioned this pull request Jul 5, 2024
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
0.8.7 Red Team
Projects
None yet
Milestone
24Q3 - G1M3 Organic Upgradability
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@koenvanderveen @madhavajay @teo-milea @eelcovdw @khoaguin