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

fix(jans-ce-setup): revert Admin UI installation option from Janssen Vm installer #691

Closed
duttarnab opened this issue Jan 24, 2022 · 1 comment
Closed

fix(jans-ce-setup): revert Admin UI installation option from Janssen Vm installer #691

duttarnab opened this issue Jan 24, 2022 · 1 comment
Assignees
@devrimyatar @moabu
Labels
kind-bug Issue or PR is a bug in existing functionality

Comments

@duttarnab
Copy link
Contributor

duttarnab commented Jan 24, 2022 •
edited
Loading

Describe the bug
Janssen is an open-source, community-governed project. There should not be an option in the Janssen Vm installer to install Admin UI which is Gluu Commercial Product.

There should not be any code to install Admin UI using Janssen Vm installer.
Please revert the changes asked in the following tickets:

  1. Admin-UI setup support in jans-setup #469
  2. Post-Installation script for installation, enable and disable plugins in admin-ui frontend #302

Note: Please keep a backup of the changes done so that we can use it to create install_gluu_flex.py script (in GluuFederation github repo) to install Admin UI on top of Janssen installation. I will raise another ticket for this task.
@duttarnab duttarnab added the kind-bug Issue or PR is a bug in existing functionality label Jan 24, 2022
@duttarnab duttarnab changed the title fix(jans-ce-setup): revert Admin UI installation from Janssen Vm installation fix(jans-ce-setup): revert Admin UI installation option from Janssen Vm installer Jan 24, 2022
devrimyatar added a commit that referenced this issue Jan 25, 2022
@devrimyatar
refactor(admin-ui): remove admin-ui (ref: #691)
4ab2ca9
yuriyz pushed a commit that referenced this issue Jan 25, 2022
@devrimyatar
fix: vm setup suse fixes (#705)
2f69a8a 
* fix: suse fixes

* refactor(admin-ui): remove admin-ui (ref: #691)

* fix: remove cb and spanner backend options for package

* refactor(ce-setup): ce-setup exclude jwt from config-cli
moabu pushed a commit that referenced this issue Jan 26, 2022
@devrimyatar @moabu
fix: vm setup suse fixes (#705)
ca87028 
* fix: suse fixes

* refactor(admin-ui): remove admin-ui (ref: #691)

* fix: remove cb and spanner backend options for package

* refactor(ce-setup): ce-setup exclude jwt from config-cli

(cherry picked from commit 2f69a8a)
moabu added a commit that referenced this issue Jan 26, 2022
@ossdhaval @moabu @devrimyatar
refactor: rename jans-ce-setup (#704)
d50da27 
* ci: rename jans-ce-setup

* refactor: rename jans-ce-setup to jans-linux-setup

* fix: vm setup suse fixes (#705)

* fix: suse fixes

* refactor(admin-ui): remove admin-ui (ref: #691)

* fix: remove cb and spanner backend options for package

* refactor(ce-setup): ce-setup exclude jwt from config-cli

(cherry picked from commit 2f69a8a)

* ci: rename jans-vm-setup to jans-linux-setup

Co-authored-by: Mohammad Abudayyeh <47318409+moabu@users.noreply.github.com>
Co-authored-by: Mustafa Baser <mbaser@mail.com>
@devrimyatar
Copy link
Contributor

Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@devrimyatar devrimyatar

@moabu moabu

Labels
kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@devrimyatar @duttarnab @moabu