[Buildbot-devel] withproperties or interpolate in ShellCommand

[Dustin J. Mitchell]

It's a little hard to tell what's going on with so much left out.

I'm surprised that you got an error about Interpolate when you were
using WithProperties.

I see that you were calling 'self.render', which suggests that this
'self.command = ' is not in the step's constructor, but in some method
that runs later, after the properties have been rendered.  Most likely
what is happening is that  Buildbot is then trying to send the
command, including the Interpolate instance, to the slave, and Twisted
is complaining that you're not allowed to send such objects over the
network.

In order to use renderables like Interpolate, you need to put them in
place before the step is started, which basically means in the step's
constructor.

Dustin

On Mon, Sep 1, 2014 at 3:48 AM, Jürgen Schulz-Brüssel
<juergen.schulz-bruessel at thinkproject.com> wrote:
>
> Hi,
>
> most of my steps are subclassed ShellCommands. I recently upgraded from
> 0.8.6 to 0.8.9 and ran in the following errors:
>
> 'Unpersistable('"'"'Unpersistable data: instance of class buildbot.process.properties.Interpolate deemed insecure'"'"')'
>
>
> my subclassed command looks like this:
>
> self.command=[data,
>                       data,
>                       self.render(WithProperties("build-%s.tar.gz",
> "revision")),
>                       ]
>
> Following the documentation I changed it to
>
> self.command=[data,
>                       data,
> Interpolate('build-%(prop:got_revision)s.tar.gz'),
>                       ]
>
> but the error is still the same. I have really no idea, what's going
> wrong. Are there any hints?
>
> Best
>      Jürgen
>
> --
>
> think project! International GmbH & Co. KG
> Jürgen Schulz-Brüssel
> Developer
>
> Phone +49 30 92 10 17-83
> Fax +49 30 92 10 17-50
> juergen.schulz-bruessel at thinkproject.com
>
> Legal information (imprint): www.thinkproject.com/en/info
>
> Rechtliche Informationen zum Absender (Impressum): www.thinkproject.com/de/info
>
>
> ------------------------------------------------------------------------------
> Slashdot TV.
> Video for Nerds.  Stuff that matters.
> http://tv.slashdot.org/
> _______________________________________________
> Buildbot-devel mailing list
> Buildbot-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/buildbot-devel