<div dir="ltr"><div dir="ltr">On Tue, Oct 4, 2022 at 5:16 AM Jan Kratochvil <<a href="mailto:jkratochvil@azul.com">jkratochvil@azul.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, 29 Sep 2022 17:45:01 +0200, Roman Marchenko wrote:<br>
> This PR provides functionality to propagate actual environment variables to<br>
> a restored process, as well as the test for this functionality.<br>
<br>
It would be nice to know what was the reason for introducing this feature.<br></blockquote><div><br></div><div>When deploying a container to K8, it's pretty common to configure the application using env vars - things like connection ports, host names, etc get injected into the container.  If a checkpoint is taken in the CI environment, we don't want to bind in those env vars as it's too early to know what the values will be for a particular deployment.  If the restores happen when the image is deployed, we need a way to inject the final configuration data (the env vars) into the restored image.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
This mail thread discusses its disadvantages but not its advantages.<br></blockquote><div><br></div><div>Less so the disadvantages and more consequences - the ability to set env vars on restore is important for containers but it has knock on effects.  Existing applications haven't been architected to expect this so there's some corner cases that need to be worked through.  </div><div><br></div><div>I think we need this kind of ability for CRaC but am now looking at how do we design it so that it limits the risk to applications, is easy to service, and has a clear model that developers and users can reason about.</div><div><br></div><div>--Dan</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
Jan<br>
<br>
</blockquote></div></div>