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

Revert websphere-liberty to using IBM JRE #1117

Merged
merged 1 commit into from
Oct 13, 2015
Merged

Revert websphere-liberty to using IBM JRE #1117

merged 1 commit into from
Oct 13, 2015

Conversation

davidcurrie
Copy link
Contributor

I'm given to understand that export controls are now in place on Docker Hub so we can revert the websphere-liberty image back to using an IBM JRE.

I'm given to understand that export controls are now in place on Docker Hub so we can revert the websphere-liberty image back to using an IBM JRE.
@yosifkit
Copy link
Member

Changeset: WASdev/ci.docker@9da673b...975e98c

LGTM, Build test of #1117; 77d8d9c (websphere-liberty):

$ url="https://raw.githubusercontent.com/docker-library/official-images/77d8d9cb881423b184a9f439aa190d8aa7787bd2/library/websphere-liberty"
$ bashbrew build "$url"
Fetching websphere-liberty (git://github.com/WASdev/ci.docker) ...
Processing websphere-liberty:kernel ...
Processing websphere-liberty:8.5.5.7-kernel ...
Processing websphere-liberty:common ...
Processing websphere-liberty:8.5.5.7-common ...
Processing websphere-liberty:webProfile6 ...
Processing websphere-liberty:8.5.5.7-webProfile6 ...
Processing websphere-liberty:webProfile7 ...
Processing websphere-liberty:8.5.5.7-webProfile7 ...
Processing websphere-liberty:javaee7 ...
Processing websphere-liberty:8.5.5.7-javaee7 ...
Processing websphere-liberty:8.5.5.7 ...
Processing websphere-liberty:8.5.5 ...
Processing websphere-liberty:latest ...
Processing websphere-liberty:beta ...
$ bashbrew list --uniq "$url" | xargs test/run.sh
testing websphere-liberty:kernel
    'utc' [1/4]...passed
    'cve-2014--shellshock' [2/4]...passed
    'no-hard-coded-passwords' [3/4]...passed
    'override-cmd' [4/4]...failed
testing websphere-liberty:common
    'utc' [1/4]...passed
    'cve-2014--shellshock' [2/4]...passed
    'no-hard-coded-passwords' [3/4]...passed
    'override-cmd' [4/4]...failed
testing websphere-liberty:webProfile6
    'utc' [1/4]...passed
    'cve-2014--shellshock' [2/4]...passed
    'no-hard-coded-passwords' [3/4]...passed
    'override-cmd' [4/4]...failed
testing websphere-liberty:webProfile7
    'utc' [1/4]...passed
    'cve-2014--shellshock' [2/4]...passed
    'no-hard-coded-passwords' [3/4]...passed
    'override-cmd' [4/4]...failed
testing websphere-liberty:javaee7
    'utc' [1/4]...passed
    'cve-2014--shellshock' [2/4]...passed
    'no-hard-coded-passwords' [3/4]...passed
    'override-cmd' [4/4]...failed
testing websphere-liberty:beta
    'utc' [1/4]...passed
    'cve-2014--shellshock' [2/4]...passed
    'no-hard-coded-passwords' [3/4]...passed
    'override-cmd' [4/4]...failed

There is at least one failure in the above build log.

@yosifkit
Copy link
Member

Disclaimer: I have no knowledge of the export controls available on Docker Hub or if they need a request to turn on.

@tianon
Copy link
Member

tianon commented Oct 13, 2015

Yeah, the diff seems fine (LGTM), but we don't know anything about export controls. 😄

@davidcurrie do you want us to wait while you verify whether anything extra needs to be done, or just merge?

@davidcurrie
Copy link
Contributor Author

davidcurrie commented Oct 13, 2015 via email

yosifkit added a commit that referenced this pull request Oct 13, 2015
Revert websphere-liberty to using IBM JRE
@yosifkit yosifkit merged commit 5ba1ef9 into docker-library:master Oct 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants