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

Merge plugin brokers (init, unified) into a single container image build #14296

Closed
nickboldt opened this issue Aug 20, 2019 · 4 comments
Closed

Merge plugin brokers (init, unified) into a single container image build #14296

nickboldt opened this issue Aug 20, 2019 · 4 comments
Labels
area/install Issues related to installation, including offline/air gap and initial setup area/plugin-broker kind/enhancement A feature request - must adhere to the feature request template. severity/P2 Has a minor but important impact to the usage or development of the system.
Milestone
7.2.0

Comments

@nickboldt
Copy link
Contributor

Is your enhancement related to a problem? Please describe.

Having two tiny containers to delete the /projects folder and then populate means 2x the number of containers to maintain, build, release to Quay or RHCC, and 2x the number to have to pull when supporting offline/airgap scenarios.

Describe the solution you'd like

We could consider merging these two tiny containers into a single one.

See also:
@nickboldt nickboldt added kind/enhancement A feature request - must adhere to the feature request template. area/plugin-broker severity/P2 Has a minor but important impact to the usage or development of the system. labels Aug 20, 2019
@nickboldt nickboldt mentioned this issue Aug 20, 2019
Merged
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Aug 20, 2019
@amisevsk
Copy link
Contributor

I'd probably do this as a part of #13452 since it resolving workspace differences would bring the brokers closer together.

@ibuziuk ibuziuk removed the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Aug 26, 2019
@ibuziuk ibuziuk added this to the 7.2.0 milestone Aug 26, 2019
@ibuziuk
Copy link
Member

ibuziuk commented Aug 26, 2019

moving to 7.2.0 for now and adding to the prioritization doc for the next sprint discussion

@ibuziuk
Copy link
Member

ibuziuk commented Aug 29, 2019

Note, that once implemented docs should be updated accordingly - https://www.eclipse.org/che/docs/che-7/che-workspaces-architecture/#che-plug-in-broker_che-workspace-components

@ibuziuk ibuziuk modified the milestones: 7.2.0, 7.1.0 Aug 29, 2019
@ibuziuk ibuziuk modified the milestones: 7.1.0, 7.2.0 Aug 29, 2019
@amisevsk amisevsk mentioned this issue Aug 29, 2019
Closed
@amisevsk amisevsk mentioned this issue Sep 9, 2019
Closed
5 tasks
@nickboldt nickboldt added the area/install Issues related to installation, including offline/air gap and initial setup label Sep 12, 2019
@ibuziuk
Copy link
Member

ibuziuk commented Sep 23, 2019

Closing. it was decided not to merge 2 images in one, but rather focus on refactoring - #14494

@ibuziuk ibuziuk closed this as completed Sep 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/install Issues related to installation, including offline/air gap and initial setup area/plugin-broker kind/enhancement A feature request - must adhere to the feature request template. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Milestone
7.2.0
Development

No branches or pull requests
4 participants
@nickboldt @ibuziuk @amisevsk @che-bot