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

CLOUD-3128 - EAP 7.2 UBI OpenShift image #234

Conversation

luck3y
Copy link
Collaborator

@luck3y luck3y commented May 2, 2019

https://issues.jboss.org/browse/CLOUD-3128
Signed-off-by: Ken Wills kwills@redhat.com

Thanks for submitting your Pull Request!

Please make sure your PR meets the following requirements:

  • Pull Request title is properly formatted: [CLOUD-XYA] Subject
  • Pull Request contains link to the JIRA issue
  • Pull Request contains description of the issue
  • Pull Request does not include fixes for issues other than the main ticket
  • Attached commits represent units of work and are properly formatted
  • You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
  • Every commit contains Signed-off-by: Your Name <yourname@example.com> - use git commit -s

rgdoliveira
rgdoliveira previously approved these changes May 3, 2019
@luck3y luck3y force-pushed the eap72-openjdk11-ubi8-dev branch 5 times, most recently from a4dda80 to db37809 Compare May 7, 2019 20:24
Copy link
Collaborator

@bstansberry bstansberry left a comment

Choose a reason for hiding this comment

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

Can this be squashed when ready?

- name: cct_module
git:
url: https://github.com/jboss-openshift/cct_module
ref: master
Copy link
Collaborator

Choose a reason for hiding this comment

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

sprint-35?

Not master.

- name: cct_module
git:
url: https://github.com/jboss-openshift/cct_module
ref: master
Copy link
Collaborator

Choose a reason for hiding this comment

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

sprint-35?

Not master.

@luck3y
Copy link
Collaborator Author

luck3y commented May 7, 2019

Yes to squash and will fix sprints.

@luck3y luck3y force-pushed the eap72-openjdk11-ubi8-dev branch from c1805ff to cce40d5 Compare May 7, 2019 23:00
@luck3y
Copy link
Collaborator Author

luck3y commented May 7, 2019

@bstansberry Fixed and squashed.

@luck3y luck3y requested a review from bstansberry May 7, 2019 23:01
@luck3y luck3y merged commit 1589e80 into jboss-container-images:eap72-openjdk11-ubi8-dev May 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants