quarkus-cafe-demo demo workload

[tosin2013]

SUMMARY

This pull request is for the quarkus-cafe-demo demo workload #2142 feature request. The Quarkus Cafe demo is an event-driven demo application built with Quarkus, AMQ Streams (Kafka), and MongoDB. The application can run locally and in OpenShift (Kubernetes.) That we would like to run on RHPDS.

ISSUE TYPE

• New role Pull Request

COMPONENT NAME

ocp-workload-quarkus-cafe-demo-role

TARGET_HOST="bastion.dev.openshift.opentlc.com"
OCP_USERNAME="openshiftuser"
ANSIBLE_USER="ec2-user"
WORKLOAD="ocp-workload-quarkus-cafe-demo-role"
GUID=1001
SSH_KEYNAME=sshkeyname.pem

Finally run the workload passing the input files as parameters:

# a TARGET_HOST is specified in the command line, without using an inventory file
ansible-playbook -i ${TARGET_HOST}, ./configs/ocp-workloads/ocp-workload.yml \
                 -e"ansible_ssh_private_key_file=/home/${ANSIBLE_USER}/.ssh/${SSH_KEYNAME}" \
                 -e"ansible_user=${ANSIBLE_USER}" \
                 -e"ocp_username=${OCP_USERNAME}" \
                 -e"ocp_workload=${WORKLOAD}" \
                 -e"guid=${GUID}" \
                 -e"ocp_user_needs_quota=true" \
                 -e"domain=cluster-${GUID}.${GUID}.example.opentlc.com" \
                 -e"ACTION=create"

ADDITIONAL INFORMATION

1. I do plan on using ansiblek8 in this role. I am currently not using that.

[stencell]

A couple things.

I stopped making individual comments because there are so many. Please replace any shell or command modules with k8s or k8s_info. I know there are a couple that you are doing things like oc new-app or oc start build and those are okay...but the rest can all use the modules.

Also, please rename this role. I am assuming this is meant for ocp4, so call it ocp4_workload_quarkus_cafe_demo and move it to the roles_ocp_workloads folder.

[tosin2013]

Ok, I will work on that.

…

On Mon, Jul 27, 2020 at 2:35 AM Nate Stephany ***@***.***> wrote: ***@***.**** requested changes on this pull request. A couple things. I stopped making individual comments because there are so many. Please replace any shell or command modules with k8s or k8s_info. I know there are a couple that you are doing things like oc new-app or oc start build and those are okay...but the rest can all use the modules. Also, please rename this role. I am assuming this is meant for ocp4, so call it ocp4_workload_quarkus_cafe_demo and move it to the roles_ocp_workloads folder. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2143 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAPCKL7YGQ47S5ZGA6BGLWDR5UN2ZANCNFSM4OUWD6TQ> .

-- <[email protected]> [image: AWS Certified SysOps Administrator - Associate]

[fridim]

@tosin2013 any update on this Pull Request ?

[tosin2013]

@tosin2013 any update on this Pull Request ?

I made the requested changes to the code. I was waiting for someone to review the pull request.

[darbyredhat]

@tosin2013 - Hi Tosin,

I'm Chád Darby, a member of the GPTE team. Nate requested that I perform a code review of your PR from a Quarkus dev perspective.

Would you be avail later this week for a BJ call to review the code? Basically a 30 min time slot where you can give me an overview of your project. I'm avail Thurs or Friday this week. Let me know what works best for you. My Google calendar is up to date. You can send me a Google calendar invite.

[darbyredhat]

I reviewed this PR from a Quarkus dev perspective. This looks good to me. Please proceed.

[stencell]

@tosin2013 This has a travis error: https://travis-ci.org/github/redhat-cop/agnosticd/builds/731410679. You might be able to fix this by merging or rebasing to development since this is quite a bit behind. Shouldn't be a problem for this workload since it is net-new.