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

Add initial scripts for Fedora 29 #212

Merged
merged 1 commit into from
Oct 23, 2018
Merged

Conversation

seemethere
Copy link
Contributor

Fedora 29 isn't scheduled to GA until 23-10-2018 but let's get ahead of the curve and start building this in our nightly pipelines.

Relates to docker/for-linux#430

Signed-off-by: Eli Uriegas eli.uriegas@docker.com

@seemethere seemethere requested a review from a team September 14, 2018 10:18
@seemethere seemethere changed the title Add initial scripts scripts for Fedora 29 Add initial scripts for building Fedora 29 Sep 14, 2018
@seemethere seemethere changed the title Add initial scripts for building Fedora 29 Add initial scripts for Fedora 29 Sep 14, 2018
@@ -69,6 +70,14 @@ fedora: fedora-28 fedora-27 fedora-26 ## build all fedora rpm packages
.PHONY: centos
centos: centos-7 ## build all centos rpm packages

.PHONY: fedora-29

Choose a reason for hiding this comment

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

Should add fedora-29 to the fedora target.
Although I'm not sure if we use that target.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We don't, we should probably just remove it

@seemethere seemethere requested a review from a team October 22, 2018 21:08
@jose-bigio
Copy link

This needs to be rebased!

Signed-off-by: Eli Uriegas <eli.uriegas@docker.com>
@seemethere
Copy link
Contributor Author

@jose-bigio rebased!

Copy link

@jose-bigio jose-bigio left a comment

Choose a reason for hiding this comment

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

LGTM

@seemethere seemethere merged commit 6b575e4 into docker:master Oct 23, 2018
@trzecieu
Copy link

Hi, Thank you for your work!
I'd like to ask if this has been effectively publish, as while testing F29 (released today, few days ago we had a change to pull the final image) I'm getting
Failed to synchronize cache for repo 'docker-ce-stable', ignoring this repo.

Cheers,

@Cyber1000
Copy link

@trzecieu
Copy link

Manage it via:

# from F29
dnf install https://download.docker.com/linux/fedora/29/x86_64/nightly/Packages/docker-ce-cli-0.0.0.20181027050239.c7c18e3-0.fc29.x86_64.rpm
# from F28, as the docker-ce requires `containerd.io` to be installed (instead of containerd in system?)
dnf install https://download.docker.com/linux/fedora/28/x86_64/stable/Packages/containerd.io-1.2.0-2.2.rc.2.1.fc28.x86_64.rpm
# from F29
dnf install https://download.docker.com/linux/fedora/29/x86_64/nightly/Packages/docker-ce-0.0.0.20181027050239.c7c18e3-0.fc29.x86_64.rpm

Looks stable, until an official repo will be available.

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