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

Custom api and worker #8693

Merged
merged 70 commits into from Apr 22, 2024
Merged

Custom api and worker #8693

merged 70 commits into from Apr 22, 2024

Conversation

madhavajay
Copy link
Collaborator

Description

Please include a summary of the change, the motivation, and any additional context that will help others understand your PR. If it closes one or more open issues, please tag them as described here.

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

IonesioJunior and others added 23 commits April 10, 2024 23:32
@IonesioJunior
Add Feature to run api endpoints in Jobs
97e964b
@madhavajay
Merge branch 'dev' into custom_api_and_worker
c69e42d
@madhavajay
Move notebook out of api folder
2b31864 
- Updating precommit to ignore
@madhavajay
Merge branch 'yash/fix-traefik-2' into custom_api_and_worker
5e03fff
@madhavajay
Merge branch 'dev' into custom_api_and_worker
b312458
@madhavajay
Added ABCMeta to serde
625d61c 
- Updated custom worker notebook with real example
@IonesioJunior
Merge branch 'dev' into custom_api_and_worker
18acf53
@IonesioJunior
Merge branch 'dev' into custom_api_and_worker
988e2af
@yashgorana
[syft] fix devspace buildkit args
5ba2053
@tcp
fix: only corepack enable is enough for pnpm
a5f964a
@IonesioJunior
Merge branch 'dev' into custom_api_and_worker
685014a
@yashgorana
[syft] temporarily add bq + dtypes for dev test
f02109f
@tcp
fix: remove platform from frontend build
fd51596
@tcp
fix: add platform settings to frontend, move target in config
c2650c6
@madhavajay
Merge branch 'dev' into custom_api_and_worker
9f7d80d
@koenvanderveen
only serialize once during upload
3348090
@koenvanderveen
Merge branch 'custom_api_and_worker' of github.com:OpenMined/PySyft i…
89c982c 
…nto custom_api_and_worker
@madhavajay
Fix linting
10cadac
@madhavajay
Renamed NodeSettings and fixed some small issues
a8ced5a 
- Added timeout while we solve bug in Admin API Job Queue
- Added sy.orchestra to fixtures for private api sync tests for jobs
- Bumped mongo version for test binary
@shubham3121
return ActionObject in call_public and call_private API endpoints
b61e6dd
@koenvanderveen
change syncing checks for jobs
3a425d1
@IonesioJunior
Fix lint issues
9c1d6c6
@IonesioJunior
Fix import compatibility with Python3.10
a37e53e
abyesilyurt and others added 19 commits April 18, 2024 13:52
@abyesilyurt
remove notebooks
bf3a999
@IonesioJunior
Merge branch 'dev' into custom_api_and_worker
9c05a4a
@IonesioJunior
Fix Linting issue
6aed590
@IonesioJunior
Fix sync unit tests
e3c74f2
@abyesilyurt
Merge branch 'custom_api_and_worker' into aziz/twin_sync_integration
e92771c
@abyesilyurt
revert change
bf3b68e
@abyesilyurt
Merge pull request #8719 from OpenMined/aziz/twin_sync_integration
91ee21d 
move custom api syncing test to integration tests
@IonesioJunior
Merge branch 'dev' into custom_api_and_worker
ee5ea80
@IonesioJunior
Uncomment line that made blob storage test fail
606c59c
@IonesioJunior
Merge branch 'dev' into custom_api_and_worker
334701d
@abyesilyurt
fix: sync private twin api job
5ba1223
@abyesilyurt
fix lint
6e0ae30
@koenvanderveen
Merge pull request #8722 from OpenMined/aziz/sync_twin_jobs
2532d39 
fix: sync private twin api job
@IonesioJunior
ADD temp workaround about the has_permission issue
37a476c
@IonesioJunior
Remove unused packages from setup.cfg
596db0d
@IonesioJunior
Fix protocol version issue
131efdc
@IonesioJunior
Replace broken hagrid flag in stack.test.integration
14fe40a
@IonesioJunior
Fix k8s test
fb86894
@IonesioJunior
Fix linting issues
38f648d
@IonesioJunior IonesioJunior changed the title WIP Custom api and worker Custom api and worker Apr 20, 2024
@madhavajay
Copy link
Collaborator Author

Solid work everyone!
Another great party PR 🎉

@IonesioJunior IonesioJunior merged commit b31a130 into dev Apr 22, 2024
35 checks passed
@IonesioJunior IonesioJunior deleted the custom_api_and_worker branch April 22, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shubham3121 shubham3121 shubham3121 left review comments

@IonesioJunior IonesioJunior IonesioJunior approved these changes

Assignees
No one assigned
Labels
0.8.7 Blue Team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@madhavajay @shubham3121 @IonesioJunior @yashgorana @tcp @koenvanderveen @abyesilyurt