Skip to content
This repository has been archived by the owner on Mar 22, 2021. It is now read-only.

Dockerfile: Update version and add verification #75

Closed
wants to merge 2 commits into from

Conversation

ashcrow
Copy link
Contributor

@ashcrow ashcrow commented Nov 15, 2017

No description provided.

Dockerfile Outdated
MAINTAINER Jonathan Lebon <[email protected]>

# NB: we install libyaml-devel so that we can use
# CSafeLoader in PyYAML (see related comment in the parser)

RUN dnf install -y \
RUN dnf install --setopt=tsflags=nodocs -y \
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think this is necessary because those options are enabled by default in the fedora:27 image:

$ sudo docker run -it registry.fedoraproject.org/fedora:27 cat /etc/dnf/dnf.conf
[main]
gpgcheck=1
installonly_limit=3
clean_requirements_on_remove=True
tsflags=nodocs

Copy link
Contributor

@miabbott miabbott left a comment

Choose a reason for hiding this comment

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

LGTM!

@jlebon
Copy link
Collaborator

jlebon commented Nov 15, 2017

This is probably fine and should probably work, but because it's the image that is run by all PAPR jobs, I need to test it out first. The way I do this currently is push to the dev branch, which creates a new dev tag, which my redhat-ci-sandbox repo is currently setup to use. Not the best, but it does the job for now (related issue #43).

@ashcrow
Copy link
Contributor Author

ashcrow commented Nov 15, 2017

@jlebon best to be safe before merging 👍

@ashcrow ashcrow closed this Oct 23, 2019
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