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

Legal stuff in docker images #169

Merged
merged 2 commits into from
Dec 13, 2017
Merged

Legal stuff in docker images #169

merged 2 commits into from
Dec 13, 2017

Conversation

shay-berman
Copy link
Contributor

@shay-berman shay-berman commented Dec 13, 2017

Add Apache LICENSE and NOTICES files into /root of the ubiquity flex and provisioner images (Dockerfile)

Note : The NOTICES file in the github repo should be locate under [scripts/notices_file_for_ibm_storage_enabler_for_containers] and inside the docker image it should be renamed as NOTICES.


This change is Reviewable

@shay-berman shay-berman self-assigned this Dec 13, 2017
@shay-berman shay-berman requested a review from alonm December 13, 2017 11:10
@coveralls
Copy link

Coverage Status

Coverage remained the same at 54.819% when pulling 795b3c9 on legal_stuff_in_docker_images into ca9d27c on dev.

@alonm
Copy link
Contributor

alonm commented Dec 13, 2017

:lgtm:


Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@shay-berman shay-berman merged commit c6dd8f8 into dev Dec 13, 2017
@shay-berman shay-berman deleted the legal_stuff_in_docker_images branch January 9, 2018 17:26
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