<div dir="auto"><div dir="auto">Hi Christoph,</div><div dir="auto"><br></div><div dir="auto">I just sent a PR which adds desired functionality for the stork driver.</div><div dir="auto">I already built docker images and pushed it to Kube-Linstor project. </div><div dir="auto">Stork scheduler is working as a charm. </div><div dir="auto"><br></div><div dir="auto">Thanks a lot for such cool opportunity!</div><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">On Mon, May 18, 2020, 09:36 kvaps &lt;<a href="mailto:kvapss@gmail.com">kvapss@gmail.com</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr"><div>Hi Christoph,</div><div><br></div><div>&gt; You are using Pireaus operator to deploy LINSTOR, right?</div><div><br></div><div>No we&#39;re using <a href="https://github.com/kvaps/kube-linstor" target="_blank" rel="noreferrer">https://github.com/kvaps/kube-linstor</a> and I&#39;m going to add support of your stork plugin for that.</div><div><br></div><div><div>I&#39;m little skeptical about using operator approach for anything else besides configuration or managing difficult application lifecycle :)<br></div><div>Instead, I like when application can be simple deployed by simple templated manifests. Kubernetes already have all needed controllers for managing it.</div><div><br></div><div>&gt;  I&#39;ll think about how to solve this more generically. Thanks for the report.</div><div><br></div><div>I&#39;m sure that adding support for the same environment variables like for linstor-csi is the easiest and better way, ideally I&#39;d like to have:</div><br></div><div>      env:<br></div><div>        - name: LINSTOR_IP<br>          value: <a href="https://linstor-controller:3371" target="_blank" rel="noreferrer">https://linstor-controller:3371</a><br>        - name: LS_USER_CERTIFICATE<br>          valueFrom:<br>            secretKeyRef:<br>              key: tls.crt<br>              name: linstor-client-tls<br>        - name: LS_USER_KEY<br>          valueFrom:<br>            secretKeyRef:<br>              key: tls.key<br>              name: linstor-client-tls<br>        - name: LS_ROOT_CA<br>          valueFrom:<br>            secretKeyRef:<br>              key: ca.crt<br>              name: linstor-client-tls<br></div><div><br></div><div>And of course do not hardcode driver with the piraeus-operator. it&#39;s better to keep the things loosely coupled with each other, and components replaceable.</div><div><br></div><div>&gt; the source code is available on GitHub, though it is somewhat hidden (in a
branch of our fork of Stork): <a href="https://github.com/LINBIT/stork/tree/linstor-driver" rel="noreferrer noreferrer" target="_blank">https://github.com/LINBIT/stork/tree/linstor-driver</a><br>
</div><div><br></div><div>Got it! Sorry, this branch had no these changes at this weekend :)<br></div><div><br></div><div><div dir="ltr"><div dir="ltr"><div>Best Regards,</div><div>Andrei Kvapil<br></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 18, 2020 at 7:59 AM Christoph Böhmwalder &lt;<a href="mailto:christoph.boehmwalder@linbit.com" target="_blank" rel="noreferrer">christoph.boehmwalder@linbit.com</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 16.05.20 23:57, kvaps wrote:<br>
&gt; Hi, I&#39;m trying to deploy stork driver using official documentation. But it is<br>
&gt; not starting due wrong controller endpoint name:<br>
&gt; <br>
&gt; time=&quot;2020-05-16T21:16:31Z&quot; level=info msg=&quot;called: Init()&quot;<br>
&gt; time=&quot;2020-05-16T21:16:31Z&quot; level=error msg=&quot;Error init&#39;ing linstor driver:<br>
&gt; error getting linstor endpoint: endpoints \&quot;linstor-op-cs\&quot; not found&quot;<br>
&gt; time=&quot;2020-05-16T21:16:31Z&quot; level=info msg=&quot;Starting stork version 2.4.0-cba53f4c&quot;<br>
&gt; time=&quot;2020-05-16T21:16:31Z&quot; level=info msg=&quot;Using driver linstor&quot;<br>
&gt; time=&quot;2020-05-16T21:16:31Z&quot; level=fatal msg=&quot;Error getting Stork Driver linstor:<br>
&gt; VolumeDriver with UID/Name: linstor not found&quot;<br>
&gt; <br>
&gt; I can&#39;t find any configuration options, also I can&#39;t find any source code to<br>
&gt; check it out.<br>
&gt; <br>
&gt; I&#39;ve tried to assign similar variables like for csi-driver (LINSTOR_IP,<br>
&gt; LS_USER_CERTIFICATE, LS_USER_KEY and LS_ROOT_CA), also try to create service<br>
&gt; with `linstor-op-cs` name, but all my attempts to start the driver were even.<br>
&gt; <br>
&gt; - kvaps<br>
&gt; <br>
&gt; _______________________________________________<br>
&gt; Star us on GITHUB: <a href="https://github.com/LINBIT" rel="noreferrer noreferrer" target="_blank">https://github.com/LINBIT</a><br>
&gt; drbd-user mailing list<br>
&gt; <a href="mailto:drbd-user@lists.linbit.com" target="_blank" rel="noreferrer">drbd-user@lists.linbit.com</a><br>
&gt; <a href="https://lists.linbit.com/mailman/listinfo/drbd-user" rel="noreferrer noreferrer" target="_blank">https://lists.linbit.com/mailman/listinfo/drbd-user</a><br>
&gt; <br>
<br>
Hi,<br>
<br>
You are using Pireaus operator to deploy LINSTOR, right?<br>
In that case hard coding &quot;linstor-op-cs&quot; was probably too naive (LINSTOR<br>
operator is our &quot;enterprise support&quot; version of Piraeus operator, so that is<br>
what we primarily test with).<br>
<br>
I&#39;ll think about how to solve this more generically. Thanks for the report.<br>
<br>
PS: the source code is available on GitHub, though it is somewhat hidden (in a<br>
branch of our fork of Stork): <a href="https://github.com/LINBIT/stork/tree/linstor-driver" rel="noreferrer noreferrer" target="_blank">https://github.com/LINBIT/stork/tree/linstor-driver</a><br>
<br>
--<br>
Christoph Böhmwalder<br>
LINBIT | Keeping the Digital World Running<br>
DRBD HA —  Disaster Recovery — Software defined Storage<br>
</blockquote></div>
</blockquote></div></div>