<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0">
<base href="https://issues.jboss.org">
<title>Message Title</title>
</head>
<body class="jira" style="color: #333333; font-family: Arial, sans-serif; font-size: 14px; line-height: 1.429">
<table id="background-table" cellpadding="0" cellspacing="0" width="100%" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt; background-color: #f5f5f5; border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt" bgcolor="#f5f5f5">
<!-- header here -->
<tbody>
<tr>
<td id="header-pattern-container" style="padding: 0px; border-collapse: collapse; padding: 10px 20px">
<table id="header-pattern" cellspacing="0" cellpadding="0" border="0" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt">
<tbody>
<tr>
<td id="header-avatar-image-container" valign="top" style="padding: 0px; border-collapse: collapse; vertical-align: top; width: 32px; padding-right: 8px" width="32"> <img id="header-avatar-image" class="image_fix" src="https://static.jboss.org/developer/gravatar/bd1946aafd256fc8796cd927befe1353?d=mm&s=48" height="32" width="32" border="0" style="border-radius: 3px; vertical-align: top"> </td>
<td id="header-text-container" valign="middle" style="padding: 0px; border-collapse: collapse; vertical-align: middle; font-family: Arial, sans-serif; font-size: 14px; line-height: 20px; mso-line-height-rule: exactly; mso-text-raise: 1px"> <a class="user-hover" rel="cmacedo86" id="email_cmacedo86" href="https://issues.jboss.org/secure/ViewProfile.jspa?name=cmacedo86" style="color:#0052cc;; color: #3b73af; text-decoration: none">Camila Macedo</a> <strong>edited a comment</strong> on <a href="https://issues.jboss.org/browse/AEROGEAR-9097" style="color: #3b73af; text-decoration: none"><img src="cid:jira-generated-image-avatar-b4241b13-b731-4a34-927e-240f5c88b8d1" height="16" width="16" border="0" align="absmiddle" alt="Task"> AEROGEAR-9097</a> </td>
</tr>
</tbody>
</table> </td>
</tr>
<tr>
<td id="email-content-container" style="padding: 0px; border-collapse: collapse; padding: 0 20px">
<table id="email-content-table" cellspacing="0" cellpadding="0" border="0" width="100%" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt; border-spacing: 0; border-collapse: separate">
<tbody>
<tr>
<!-- there needs to be content in the cell for it to render in some clients -->
<td class="email-content-rounded-top mobile-expand" style="padding: 0px; border-collapse: collapse; color: #ffffff; padding: 0 15px 0 16px; height: 15px; background-color: #ffffff; border-left: 1px solid #cccccc; border-top: 1px solid #cccccc; border-right: 1px solid #cccccc; border-bottom: 0; border-top-right-radius: 5px; border-top-left-radius: 5px; height: 10px; line-height: 10px; padding: 0 15px 0 16px; mso-line-height-rule: exactly" height="10" bgcolor="#ffffff"> </td>
</tr>
<tr>
<td class="email-content-main mobile-expand " style="padding: 0px; border-collapse: collapse; border-left: 1px solid #cccccc; border-right: 1px solid #cccccc; border-top: 0; border-bottom: 0; padding: 0 15px 0 16px; background-color: #ffffff" bgcolor="#ffffff">
<table class="page-title-pattern" cellspacing="0" cellpadding="0" border="0" width="100%" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt">
<tbody>
<tr>
<td style="vertical-align: top;; padding: 0px; border-collapse: collapse; padding-right: 5px; font-size: 20px; line-height: 30px; mso-line-height-rule: exactly" class="page-title-pattern-header-container"> <span class="page-title-pattern-header" style="font-family: Arial, sans-serif; padding: 0; font-size: 20px; line-height: 30px; mso-text-raise: 2px; mso-line-height-rule: exactly; vertical-align: middle"> <a href="https://issues.jboss.org/browse/AEROGEAR-9097" style="color: #3b73af; text-decoration: none">Re: Confirm that the user only needs to call one install</a> </span> </td>
</tr>
</tbody>
</table> </td>
</tr>
<tr>
<td id="text-paragraph-pattern-top" class="email-content-main mobile-expand comment-top-special-margin comment-top-pattern" style="padding: 0px; border-collapse: collapse; border-left: 1px solid #cccccc; border-right: 1px solid #cccccc; border-top: 0; border-bottom: 0; padding: 0 15px 0 16px; background-color: #ffffff; border-bottom: none; padding-bottom: 0" bgcolor="#ffffff">
<table class="text-paragraph-pattern" cellspacing="0" cellpadding="0" border="0" width="100%" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt; font-family: Arial, sans-serif; font-size: 14px; line-height: 20px; mso-line-height-rule: exactly; mso-text-raise: 2px">
<tbody>
<tr>
<td class="text-paragraph-pattern-container mobile-resize-text " style="padding: 0px; border-collapse: collapse; padding: 0 0 10px 0; padding-top: 10px"> <span class="diffcontext">Hi [~lfitzgerald],<br><br>The App is one CR as the Database is another. Note that each CR means one type of resource. This project was developed following exactly the instructions of the Getting Started and the operator-framework doc</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> as</span> <span class="diffaddedchars" style="background-color:#ddfade;">.<br><br>If you check in</span> <span class="diffcontext"> the</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> documentation provided</span> <span class="diffaddedchars" style="background-color:#ddfade;"> docs and</span> <span class="diffcontext"> in the</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> training</span> <span class="diffaddedchars" style="background-color:#ddfade;"> examples you will see</span> <span class="diffcontext"> that</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> the team attend over</span> <span class="diffaddedchars" style="background-color:#ddfade;"> all operators following</span> <span class="diffcontext"> this</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> subject</span> <span class="diffaddedchars" style="background-color:#ddfade;"> idea/approach which shows indeed what is recommended to get done as a good practice anyway</span> <span class="diffcontext">.<br><br></span> <span class="diffaddedchars" style="background-color:#ddfade;">{panel:title=Examples}<br><br></span> <span class="diffcontext">Note that the way that it is now with one type for the DB and another for the APP we are following the same approach of Prometheus for example</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> which I understand that it is doing as it is recommended to get done</span> <span class="diffcontext">. The Prometheus has one type/ CR for each thing. See here <https://github.com/coreos/prometheus-operator>. <br><br></span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;">If you check in the docs and in the examples you will see that all operators following this idea/approach that we are doing here and not just Prometheus which definitely is a great example. <br><br></span> <span class="diffcontext">This one has the service and database as us: <br>* https://operatorhub.io/operator/alpha/aqua-operator.v0.0.1 ( AquaDatabase CR, AquaServer CR, AquaGateway CR and etc ) <br>* https://github.com/aquasecurity/aqua-operator/tree/master/deploy/crds<br><br>This one is from AWS in HELM and has one CR for each thing as well ( DynamoDB, ECRRepository and etc .. )<br>* https://github.com/awslabs/aws-service-operator/tree/master/examples<br><br>* Etcd is another excellent project to follow an example and is doing as us one CR for each thing ( etcd Cluster, etcd Backup, etcd Restore )<br>* https://github.com/coreos/etcd-operator/tree/master/example<br>* https://operatorhub.io/operator/clusterwide-alpha/etcdoperator.v0.9.4-clusterwide<br><br>Please, feel free to check the operators available in https://operatorhub.io/ to validate this approach as its documents. <br></span> <span class="diffaddedchars" style="background-color:#ddfade;">{panel}</span> <span class="diffcontext"><br></span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;">Also</span> <span class="diffaddedchars" style="background-color:#ddfade;"><br>Besides</span> <span class="diffcontext">,</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> if we</span> <span class="diffaddedchars" style="background-color:#ddfade;"> be good practice and which shows recommended and followed for all regards</span> <span class="diffcontext"> create</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> a</span> <span class="diffaddedchars" style="background-color:#ddfade;"> one type/</span> <span class="diffcontext"> CR for</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> all</span> <span class="diffaddedchars" style="background-color:#ddfade;"> each thing I am against to change</span> <span class="diffcontext"> it</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> will not be a good approach at all</span> <span class="diffcontext"> for</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> many</span> <span class="diffaddedchars" style="background-color:#ddfade;"> other</span> <span class="diffcontext"> reasons as</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> for example:<br><br>* We will define</span> <span class="diffaddedchars" style="background-color:#ddfade;"> well. It is clear</span> <span class="diffcontext"> that</span> <span class="diffaddedchars" style="background-color:#ddfade;"> we have</span> <span class="diffcontext"> the</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> type MSS is</span> <span class="diffaddedchars" style="background-color:#ddfade;"> intention to remove</span> <span class="diffcontext"> the</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> project + database, for me conceptual it not make sense</span> <span class="diffaddedchars" style="background-color:#ddfade;"> DB in the future and use a community operator then following some extra argumentations</span> <span class="diffcontext">. <br></span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;">* The way that</span> <span class="diffaddedchars" style="background-color:#ddfade;"><br>As</span> <span class="diffcontext"> it</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> is now we</span> <span class="diffaddedchars" style="background-color:#ddfade;"> was implemented/designed: <br><br>* We</span> <span class="diffcontext"> can just not install the type DB and use a community operator if we wish<br>*</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> The wat that is now we</span> <span class="diffaddedchars" style="background-color:#ddfade;"> We</span> <span class="diffcontext"> can easily extract the DB and create a new operator just for it<br>*</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> if</span> <span class="diffaddedchars" style="background-color:#ddfade;"> If</span> <span class="diffcontext"> we would like to remove the DB from the project then it is very very easy since is very clear what should be removed or not when we have a specific type for each thing.<br><br>In order to attend the need</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> and</span> <span class="diffaddedchars" style="background-color:#ddfade;"> to</span> <span class="diffcontext"> make the things easier for the users and us we impl the make commands. By the make commands you can, for example, run `make create-all` and it will create the namespace, apply the CRs, the roles and the service account</span> <span class="diffaddedchars" style="background-color:#ddfade;"> and etc</span> <span class="diffcontext">.</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> <br></span> <span class="diffaddedchars" style="background-color:#ddfade;">..</span> <span class="diffcontext"><br></span> <span class="diffaddedchars" style="background-color:#ddfade;"><br></span> <span class="diffcontext">Also, we can</span> <span class="diffremovedchars" style="background-color:#ffe7e7;text-decoration:line-through;"> just do the required impl to make</span> <span class="diffaddedchars" style="background-color:#ddfade;"> put</span> <span class="diffcontext"> our project available in the operatorHub as well which will allow the users to install our product by click and a beautiful interface. See the PR: <https://github.com/aerogear/mobile-security-service-operator/pull/15> It is already doing it, just need some changes/fixes to get done. <br><br></span> <span class="diffaddedchars" style="background-color:#ddfade;">PS.:</span> <span class="diffcontext">I</span> <span class="diffaddedchars" style="background-color:#ddfade;"> understand that RHMI will require the same files which are needed to publish the operator in the https://operatorhub.io/. <br><br>I</span> <span class="diffcontext"> hope that this info clarifies the reasons for it got done in this way and allow us to close this task. <br><br>c/c [~dffrench] @chfoley@redhat.com <br></span> </td>
</tr>
</tbody>
</table> </td>
</tr>
<tr>
<td class="email-content-main mobile-expand " style="padding: 0px; border-collapse: collapse; border-left: 1px solid #cccccc; border-right: 1px solid #cccccc; border-top: 0; border-bottom: 0; padding: 0 15px 0 16px; background-color: #ffffff" bgcolor="#ffffff">
<table id="actions-pattern" cellspacing="0" cellpadding="0" border="0" width="100%" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt; font-family: Arial, sans-serif; font-size: 14px; line-height: 20px; mso-line-height-rule: exactly; mso-text-raise: 1px">
<tbody>
<tr>
<td id="actions-pattern-container" valign="middle" style="padding: 0px; border-collapse: collapse; padding: 10px 0 10px 24px; vertical-align: middle; padding-left: 0">
<table align="left" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt">
<tbody>
<tr>
<td class="actions-pattern-action-icon-container" style="padding: 0px; border-collapse: collapse; font-family: Arial, sans-serif; font-size: 14px; line-height: 20px; mso-line-height-rule: exactly; mso-text-raise: 0; vertical-align: middle"> <a href="https://issues.jboss.org/browse/AEROGEAR-9097#add-comment" target="_blank" title="Add Comment" style="color: #3b73af; text-decoration: none"> <img class="actions-pattern-action-icon-image" src="cid:jira-generated-image-static-comment-icon-168ced2a-bb55-436b-a401-11593e01ac62" alt="Add Comment" title="Add Comment" height="16" width="16" border="0" style="vertical-align: middle"> </a> </td>
<td class="actions-pattern-action-text-container" style="padding: 0px; border-collapse: collapse; font-family: Arial, sans-serif; font-size: 14px; line-height: 20px; mso-line-height-rule: exactly; mso-text-raise: 4px; padding-left: 5px"> <a href="https://issues.jboss.org/browse/AEROGEAR-9097#add-comment" target="_blank" title="Add Comment" style="color: #3b73af; text-decoration: none">Add Comment</a> </td>
</tr>
</tbody>
</table> </td>
</tr>
</tbody>
</table> </td>
</tr>
<!-- there needs to be content in the cell for it to render in some clients -->
<tr>
<td class="email-content-rounded-bottom mobile-expand" style="padding: 0px; border-collapse: collapse; color: #ffffff; padding: 0 15px 0 16px; height: 5px; line-height: 5px; background-color: #ffffff; border-top: 0; border-left: 1px solid #cccccc; border-bottom: 1px solid #cccccc; border-right: 1px solid #cccccc; border-bottom-right-radius: 5px; border-bottom-left-radius: 5px; mso-line-height-rule: exactly" height="5" bgcolor="#ffffff"> </td>
</tr>
</tbody>
</table> </td>
</tr>
<tr>
<td id="footer-pattern" style="padding: 0px; border-collapse: collapse; padding: 12px 20px">
<table id="footer-pattern-container" cellspacing="0" cellpadding="0" border="0" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt">
<tbody>
<tr>
<td id="footer-pattern-text" class="mobile-resize-text" width="100%" style="padding: 0px; border-collapse: collapse; color: #999999; font-size: 12px; line-height: 18px; font-family: Arial, sans-serif; mso-line-height-rule: exactly; mso-text-raise: 2px"> This message was sent by Atlassian Jira <span id="footer-build-information">(v7.12.1#712002-<span title="609a50578ba6bc73dbf8b05dddd7c04a04b6807c" data-commit-id="609a50578ba6bc73dbf8b05dddd7c04a04b6807c}">sha1:609a505</span>)</span> </td>
<td id="footer-pattern-logo-desktop-container" valign="top" style="padding: 0px; border-collapse: collapse; padding-left: 20px; vertical-align: top">
<table style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt">
<tbody>
<tr>
<td id="footer-pattern-logo-desktop-padding" style="padding: 0px; border-collapse: collapse; padding-top: 3px"> <img id="footer-pattern-logo-desktop" src="https://issues.jboss.org/images/mail/atlassian-email-logo.png" alt="Atlassian logo" title="Atlassian logo" width="191" height="24" class="image_fix"> </td>
</tr>
</tbody>
</table> </td>
</tr>
</tbody>
</table> </td>
</tr>
</tbody>
</table>
</body>
</html>