<div dir="ltr"><div>Yeah. There are some discussions about the needs for this change:</div><div>- <a href="https://github.com/LINBIT/saas/pull/1#issuecomment-1152117095">https://github.com/LINBIT/saas/pull/1#issuecomment-1152117095</a></div><div>- <a href="https://github.com/piraeusdatastore/piraeus-operator/issues/315">https://github.com/piraeusdatastore/piraeus-operator/issues/315</a></div><div><br></div><div>In short: we're running SPAAS locally to allow our clients with isolated environments be able to build DRBD module in their clusters without need of spatch dependency as it adds additional 523 megabytes to every container.<br></div><div><br></div><div>> What is the purpose of this definition in the top-level Makefile? As far as I can see, it is only needed in the other one.<br>> <br>> I guess it serves a documentation purpose, showing what command line variables can be set. That is presumably the point of the SPAAS variable in the top-level makefile. In that case, explicitly pass it down like the SPAAS variable for consistency.</div><div><br></div><div>Yes, I just added it into the same files where the SPAAS variable is specified.</div><div>I think it is good to have it described somewhere, but it always has a default value and it can work without specifying it in Makefiles at all.<br></div><div><br></div><div><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><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 Wed, Jun 15, 2022 at 6:32 PM Joel Colledge <<a href="mailto:joel.colledge@linbit.com">joel.colledge@linbit.com</a>> 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">Thanks for the contribution. Seems like a reasonable idea to me.<br>
<br>
> diff --git a/Makefile b/Makefile<br>
> index 87c29518..37d7851d 100644<br>
> --- a/Makefile<br>
> +++ b/Makefile<br>
> @@ -37,6 +37,7 @@ DOCKERIMAGESTARGETS = $(addprefix dockerimage.,$(DOCKERIMAGES))<br>
> # Use the SPAAS (spatch as a service) online service<br>
> # Have this as make variable for distributions.<br>
> SPAAS ?= true<br>
> +SPAAS_URL ?= <a href="https://drbd.io:2020" rel="noreferrer" target="_blank">https://drbd.io:2020</a><br>
<br>
What is the purpose of this definition in the top-level Makefile? As<br>
far as I can see, it is only needed in the other one.<br>
<br>
I guess it serves a documentation purpose, showing what command line<br>
variables can be set. That is presumably the point of the SPAAS<br>
variable in the top-level makefile. In that case, explicitly pass it<br>
down like the SPAAS variable for consistency.<br>
<br>
Best regards,<br>
Joel<br>
</blockquote></div>