Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

PAI webportal plugin, make marketplace a plugin. #1700

Merged
merged 5 commits into from
Dec 27, 2018

Conversation

Gerhut
Copy link
Member

@Gerhut Gerhut commented Nov 14, 2018

Some further works to do

  1. The WEBPORTAL_PLUGINS environment variable needs to be a JSON formatted list, currently we can only serialize the list to JSON string in paiObjectModel.py. I will move the serialization to template webportal.yaml.template when jinja's tojson filter (introduced in jinja 2.9, currently we are using 2.8) is available to use. (Perhaps mounting a configmap is better.)
  2. The built-in plugins (now only marketplace) is hard-coded in paiObjectModel.py, will move to web portal's default config when new object model is landed.
  3. The GitHub owner / repository / path config now belongs to REST server, but not web portal / web portal plugin. So currently I remain the config as it was. Moving the config from REST server to web portal needs more designs in Marketplace.

@coveralls
Copy link

coveralls commented Nov 14, 2018

Coverage Status

Coverage remained the same at 51.687% when pulling 00570f6 on qixcheng/webportal/plugin into 6214662 on master.

@Gerhut Gerhut force-pushed the qixcheng/webportal/plugin branch from 8e0c5d0 to 1a900eb Compare November 15, 2018 09:28
@Gerhut Gerhut changed the title [WIP] PAI webportal plugin, make marketplace a plugin. PAI webportal plugin, make marketplace a plugin. Nov 17, 2018
@Gerhut Gerhut force-pushed the qixcheng/webportal/plugin branch from 1a900eb to 43a413f Compare November 19, 2018 06:25
@Gerhut Gerhut added this to the current release milestone Nov 21, 2018
@Gerhut Gerhut force-pushed the qixcheng/webportal/plugin branch from 43a413f to 271f408 Compare December 7, 2018 09:56
@Gerhut
Copy link
Member Author

Gerhut commented Dec 7, 2018

Rebased on current object model

@Gerhut Gerhut force-pushed the qixcheng/webportal/plugin branch from 271f408 to 50f0e43 Compare December 14, 2018 07:14
@Gerhut Gerhut requested a review from scarlett2018 December 19, 2018 14:34
@Gerhut Gerhut merged commit 22a0e1b into master Dec 27, 2018
wangcan0329 added a commit that referenced this pull request Jan 4, 2019
hao1939 pushed a commit that referenced this pull request Jan 8, 2019
* Add plugin support

* Rename env.js to env.js.template to avoid eslint

* Make marketplace a web portal plugin

* web portal: recover plugin doc
@Gerhut Gerhut deleted the qixcheng/webportal/plugin branch January 18, 2019 07:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants