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.

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

Updates from CRW 2.2 build scripts #482

Merged
merged 7 commits into from
Jun 4, 2020

Conversation

nickboldt
Copy link
Contributor

@nickboldt nickboldt commented May 28, 2020

Updates from CRW 2.2 build scripts (to better align up and downstream repos)

Change-Id: I732f8d891bded2400bac635c43dd3b212e7b13cc
Signed-off-by: nickboldt nboldt@redhat.com

@nickboldt nickboldt changed the title rename build/dockerfiles/fedora.repo to... Updates from CRW 2.2 build scripts May 28, 2020
@che-bot
Copy link
Contributor

che-bot commented May 28, 2020

Happy path tests passed.

@nickboldt
Copy link
Contributor Author

shellcheck should pass after #488 (or at least only fail for these files and not unrelated things)

@ericwill
Copy link
Contributor

ericwill commented Jun 3, 2020

I think with #481 merged this one can go in now too, right @nickboldt ?

@@ -0,0 +1,11 @@
[centos-8-for-appstream]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as you're moving files, maybe we should move specific files for rhel.Dockerfile to rhel folder ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there's a lot of logic in place relying on *.repo files in build/dockerfiles/ in various repos. Yes, we could move stuff around again but then we'd have to it for plugin reg, devfile reg, plugin brokers, machine exec, che/crw server, che/crw-theia... Not sure it's worth the churn right now.

nickboldt added 6 commits June 4, 2020 11:01
Change-Id: I777b9353d9634f810598effa51ea88f26983a171
Signed-off-by: nickboldt <nboldt@redhat.com>
Change-Id: Ie8df16a14c44a4ccdaedeb02e045a504de18385a
Signed-off-by: nickboldt <nboldt@redhat.com>
… devfile reg script and downstream version

Change-Id: Ibb2ae67ac5442e85b9aceba4500b25deb457b9aa
Signed-off-by: nickboldt <nboldt@redhat.com>
…/dockerfiles/content_sets_centos8.repo and include baseos repo too

Change-Id: Ie2cede8bbf5f9d55089a6e2acd24b8733e1b6003
Signed-off-by: nickboldt <nboldt@redhat.com>
…s_fedora30.repo for consistency/accuracy

Change-Id: I732f8d891bded2400bac635c43dd3b212e7b13cc
Signed-off-by: nickboldt <nboldt@redhat.com>
…f SCRIPT_DIR

Change-Id: I12a670025b18ace6a64f1a2a7201160d751bd594
Signed-off-by: nickboldt <nboldt@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Jun 4, 2020

Happy path tests passed.

Change-Id: I6ce96afa77d9791b74dd1d638cbcbcf0c23aff1e
Signed-off-by: nickboldt <nboldt@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Jun 4, 2020

Happy path tests passed.

3 similar comments
@che-bot
Copy link
Contributor

che-bot commented Jun 4, 2020

Happy path tests passed.

@che-bot
Copy link
Contributor

che-bot commented Jun 4, 2020

Happy path tests passed.

@che-bot
Copy link
Contributor

che-bot commented Jun 4, 2020

Happy path tests passed.

@nickboldt nickboldt merged commit ba10bf6 into eclipse-che:master Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants