-
Notifications
You must be signed in to change notification settings - Fork 28
Additions to director API for reverse proxy #363
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
Milestone
Comments
pcrespov
pushed a commit
to pcrespov/osparc-simcore
that referenced
this issue
Nov 26, 2018
pcrespov
added a commit
that referenced
this issue
Nov 29, 2018
- implements resolution policy using director (extends responses/parameters of director API to do so) - director starts backend dynamic services on a basepath - implements new routing in front-end (UI) - checks jupyter redirection - closes #207 * webserver.director subsystem: - added a function to retrieve a client provided the application - added notes on a bug * Modifies director API specs. Implementation expected in #363 * running_services get now returns also the service and its key/version * Fixes webserver director.handlers to work according to requirements * automatically create the resolved oas file * updated generator version * updated director sdk * updated director openapi * Implemented ServiceResolutionPolicy with director in application_proxy Activated setup_app_proxy in appliction * Cleanup director tests while reviewing them * Fixes duplicated storage in docker-compose * Minor cleanup and adds apihub dependency to storage * Modified director APi specs * default is empty basepath * added apihub * start notebook with a base path * version bump * added SIMCORE_NODE_BASEPATH * fixed copy of custom.js to set notebook in single-tab mode * Prepared integration-test for reverse proxy (run by hand) - recreates docker-compose and config using existing files and environs * websocket also on alternative basepath * application-proxy: - removed reference to director_sdk (using now aiohttp client) - fixed urls
sanderegg
pushed a commit
that referenced
this issue
Nov 30, 2018
* Fixes typos in client after bad merge * webserver.director subsystem: - added a function to retrieve a client provided the application - added notes on a bug * WIP: building coupling between revers_proxy and dicector subsystems * Modifies director API specs. Implementation expected in #363 * cleanup and naming services * WIP: implementing service resolution policy with director sdk * running_services get now returns also the service and its key/version * modified tests according to requirements * modified handlers to work according to requirements * pylint added a main label when more than one docker service is composing a node implemented get_service_details * updated server information * automatically create the resolved oas file * updated generator version * updated director sdk * updated director openapi * updated version check * fixed regex error in client code * updated sample * revert change * formatted director.producer to ready easier * Implemented ServiceResolutionPolicy with director in application_proxy Activated setup_app_proxy in appliction * Minor * Cleanup dev requirements * Cleanup director tests while reviewing them * Fixes duplicated storage in docker-compose * Minor cleanup and adds apihub dependency to storage * Modified director APi specs * WIP: application proxy implementation using new director api * default is empty basepath * generated client for new api with basepath * updated director api server side * Minor cleanup changed APP_SCHEMA by app_schema removed comments so we can pass to xargs in command line * update test for director * updated tests * all tests passing * update to the api * fixed not getting service correctly * WIP: implementing integration tests for reverse proxy * added apihub * start notebook with a base path * version bump * added SIMCORE_NODE_BASEPATH * fixed copy of custom.js to set notebook in single-tab mode * fixed alternative basepath * adding missing veriable and baseurl env * refactoring adding base url using async * added config file moved s4l stuff in s4l_utils * Prepared integration-test for reverse proxy - recreates docker-compose and config using existing files and environs - tests fail * websocket also on alternative basepath * duplicate key * pylint * fix failing test * version bump * empty basepath default * handle error * fix catch * fix default env * final fix * Cleanup: - removed pdb breakpoint - added disabling login option for testing - cleanup tests * Fixed webserver.director handlers * Minor * Using aiohttp client session instead of sdk * Minor * Added makefile to build swarm and tooling * webserver.director found director api endpoint upon setup * application-proxy: - removed reference to director_sdk (using now aiohttp client) - fixed urls * materialDB for LF updated * Fixes responses from director * Fixes on jupyter handlers * application_proxy: service url considers when the app is in swarm or not * tests pass using a sleep to wait the service to start * Added 'fake' data for LF MaterialDB (getItem & getItemList) * s4l converter script added * linting * Disableds proxy if director is disabled * Added 'fake' data for LF Modeler (getItemList) * Minor * Added 'fake' data mapper for Material2Entities * fake data updated * s4l converter extended * fixes eslint * Fixes test-failure in storage * Minor * Adds stdin_open and tty options in docker-compose.devel to enable debugging upon attach * Fixes oas_repo config for docker-dev. repo root does not exists inside the container! * key: label instead of key: value * FakeData moved to frontend * return sorted getItemList for materialDB and modeler * Minor bug in weberver.director subsystem * Front-end now requests interactive services via the reverse proxy * minor * minor * convert APITree into VirtualTree Model * NodeOutputTreeItem added * modeler-neuron fake data added * neuroman default inputs fake data added * qxapp.dev.fake.lf.Data added * minor * All fake data well refactored * New Project Thumbnails * sensors added to LF * sensors added to LF * Fixes reverse_proxy jupyter handler * Fixes webserver.projects handler to retrieve template projects * Minor cleanup * template projects fix * fake data update * minor * this was missing in the patch * clean up * Porjects live in the server * minor * more LF * Modeler has all entities * Neuron simulator fake data added * send nodeKey instead of nodeId for gettings items * send nodeKey instead of nodeId for gettings items (2) * ups * Wokring on Neuron defaults fake data * Nueron defaultIInputs added * more fake data * Mapper default values in and working * Default mapperInputValues get form * test updated * LF sensor can be dropped in Neuron sources * Neuroman list updated * version dump
pcrespov
added a commit
that referenced
this issue
Nov 30, 2018
Some missing parts in PR #371 that i did not push to origin. Includes -updates front-end path to dynamic services - fixes in jupyter notebook * webserver.director subsystem: - added a function to retrieve a client provided the application - added notes on a bug * Modifies director API specs. Implementation expected in #363 * cleanup and naming services * running_services get now returns also the service and its key/version * modified tests according to requirements * modified handlers to work according to requirements * updated server information * automatically create the resolved oas file * updated generator version * updated director sdk * updated director openapi * updated version check * fixed regex error in client code * updated sample * revert change * formatted director.producer to ready easier * Implemented ServiceResolutionPolicy with director in application_proxy Activated setup_app_proxy in appliction * Minor * Cleanup dev requirements * Cleanup director tests while reviewing them * Fixes duplicated storage in docker-compose * Minor cleanup and adds apihub dependency to storage * Modified director APi specs * default is empty basepath * generated client for new api with basepath * updated director api server side * Minor cleanup changed APP_SCHEMA by app_schema removed comments so we can pass to xargs in command line * update test for director * updated tests * update to the api * fixed not getting service correctly * added apihub * start notebook with a base path * version bump * added SIMCORE_NODE_BASEPATH * fixed copy of custom.js to set notebook in single-tab mode * fixed alternative basepath * adding missing veriable and baseurl env * refactoring adding base url using async * added config file moved s4l stuff in s4l_utils * Prepared integration-test for reverse proxy - recreates docker-compose and config using existing files and environs - tests fail * websocket also on alternative basepath * duplicate key * pylint * fix failing test * version bump * empty basepath default * handle error * fix catch * fix default env * final fix * Cleanup: - removed pdb breakpoint - added disabling login option for testing - cleanup tests * Fixed webserver.director handlers * Using aiohttp client session instead of sdk * Added makefile to build swarm and tooling * webserver.director found director api endpoint upon setup * application-proxy: - removed reference to director_sdk (using now aiohttp client) - fixed urls * Fixes responses from director * Fixes on jupyter handlers * application_proxy: service url considers when the app is in swarm or not * tests pass using a sleep to wait the service to start * Disableds proxy if director is disabled * Adds stdin_open and tty options in docker-compose.devel to enable debugging upon attach * Fixes oas_repo config for docker-dev. repo root does not exists inside the container! * Front-end now requests interactive services via the reverse proxy * Fixes reverse_proxy jupyter handler * Fixes webserver.projects handler to retrieve template projects
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
required by #207
The text was updated successfully, but these errors were encountered: