Open Stack

On Wed, Jun 29, 2016 at 02:59:45PM +0200, Dmitry Tantsur wrote:
> On 06/28/2016 01:37 PM, Erno Kuvaja wrote:
> > TL;DR
> > 
> > Makes absolutely sense to run file backend on single node undercloud at CI.
> > 
> > Few more comments inline.
> > 
> > On Mon, Jun 27, 2016 at 8:49 PM, Emilien Macchi <emilien at redhat.com> wrote:
> > > On Mon, Jun 27, 2016 at 3:46 PM, Clay Gerrard <clay.gerrard at gmail.com> wrote:
> > > > There's probably some minimal gain in cross compatibility testing to
> > > > sticking with the status quo.  The Swift API is old and stable, but I
> > > > believe there was some bug in recent history where some return value in
> > > > swiftclient changed from a iterable to a generator or something and some
> > > > aggressive non-duck type checking broke something somewhere....
> > > > 
> > > > I find that bug reports sorta interesting, the reported memory pressure
> > > > there doesn't make sense.  Maybe there's some non-
> > > > essential middleware configured on that proxy that's causing the workers to
> > > > bloat up like that?
> > > 
> > > Swift proxy pipeline:
> > > pipeline = catch_errors healthcheck cache ratelimit bulk tempurl
> > > formpost authtoken keystone staticweb proxy-logging proxy-server
> > 
> > Some things I do not think we benefit having there if we want to
> > experiment still with swift in undercloud:
> 
> I hope we're not removing it completely...

No, definitely not - we require Swift for several things other than backing
glance, including:

- Storing introspection data from ironic-inspector
- Signals/Metadata-polling for Heat using the tempurl transport
- Mistral deployment workflows, where plans are pushed into swift

> > staticweb - do we need containers being presented as webpages?
> > tempurl - Id assume we can expect the user having access the needed
> > objects with their own credentials.
> 
> Please leave it there, we need it to support agent_* family of ironic
> drivers.

Yes, we need tempurl for heat metadata/signals and also upload of artefacts
such as puppet modules to the nodes, so we definitely need tempurl.

Steve