metron-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kylerichardson <...@git.apache.org>
Subject [GitHub] incubator-metron pull request #409: METRON-644 RPM builds only work with Doc...
Date Tue, 03 Jan 2017 18:30:52 GMT
GitHub user kylerichardson opened a pull request:

    https://github.com/apache/incubator-metron/pull/409

    METRON-644 RPM builds only work with Docker for Mac

    Currently using the rpm-docker image to build the project RPMs will only succeed using
Docker for Mac due to the way it handles permissions. When using docker-machine/boot2docker,
"Bad owner/group" build errors occur due to the SPEC and SOURCE files having an owner uid
which does not map to an existing user within the container.
    
    Some background. When using Docker for Mac any volume mounted from the host into the container
is munged into being owned by the running user within the container. Hence there are no permissions
issues like we see with docker-machine/boot2docker.
    
    Modified the build.sh script to ensure the owner uid of the metron.spec file maps to an
existing user in the container and that the rpms post-build are owned by the same user.
    
    Tested on Docker for Mac, docker-machine/boot2docker, and a local docker-engine on Linux.
RPMs build as expected in each case.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/kylerichardson/incubator-metron METRON-644

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-metron/pull/409.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #409
    
----
commit b5d2168030e1acd9df490991f2a7f74124d71ba7
Author: kylerichardson <kylerichardson2@gmail.com>
Date:   2017-01-03T01:34:05Z

    METRON-644 Update rpm-docker build.sh
    
    Updated the script to account for non-existent user file ownership within
    the container.

commit f00bbd385e505128a8dcbc674e4630aadecabf92
Author: kylerichardson <kylerichardson2@gmail.com>
Date:   2017-01-03T04:18:22Z

    METRON-644 Correction based on Docker for Mac testing
    
    Using Docker for Mac, the id command returns non-zero for
    UID=0. Account for this behavior by ignoring UID=0 as root
    will always exist.

commit 9683c3ec69d4982b9c86558e63194ba0b6382b85
Author: Kyle Richardson <kylerichardson@users.noreply.github.com>
Date:   2017-01-03T18:19:00Z

    Update comments

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message