[libvirt] [jenkins-ci PATCH 3/5] guests: Make PERL5LIB dynamic

Daniel P. Berrangé berrange at redhat.com
Tue Jun 5 11:13:42 UTC 2018


On Tue, Jun 05, 2018 at 01:10:45PM +0200, Andrea Bolognani wrote:
> On Tue, 2018-06-05 at 11:42 +0100, Daniel P. Berrangé wrote:
> > On Mon, May 21, 2018 at 06:53:25PM +0200, Andrea Bolognani wrote:
> > > Instead of having to specify this separately for every
> > > guest due to minor differences in the environment, figure
> > > it out at login time by asking Perl itself for the
> > > information we need.
> > 
> > We still have the problem that any time a package update is done it
> > might pull in a newer Perl, which will change the path, but our
> > Jenkins slave never picks it up until the VM is rebooted.
> 
> Is that the case, though? I'm not entirely clear on the lifecycle,
> but it might very well be the case that the Jenkins agent spawns
> a separate session for each job, in which case changes to the shell
> profile would be picked up immediately.

Since we move env variables out of jenkins host config and into
the ansible host config, any time we've made env variable changes
we've had to restart the agent or host to pick it up.

> > I wonder if we can solve this by instead of putting all these
> > env vars in the main bash profile, we create a
> > 
> >    'jenkins-ci.sh'
> > 
> > that contains all the env variables. We can then source that
> > from the bash profile, and also source it as the first thing
> > we do in every build job too.
> 
> If we really need to do that, might as well source ~/.bashrc right
> at the start of every build job instead of having a separate
> script: the only unnecessary bits are setting up the prompt and
> bash completion, but there's no harm in having them in there.

Oh true, yes, we can do that.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




More information about the libvir-list mailing list