Comment 2 for bug 2039678

Revision history for this message
Alex Kavanagh (ajkavanagh) wrote :

> > The error is about missing vip a vip config, but it shouldn't be a critical error. Instead, the status can be "blocked".

> For the record, a single unit without vip is totally fine from a deployment point of view. It's not for production usage, but it's well suited for testing purposes. So it doesn't have to be even "blocked" when hacluster relation is not there yet.

Yes, totally agree. The current code for the vips() is broken, in that it should default to a string, not None:

    @property
    def vips(self):
        return self.config.get('vip').split()

e.g. just changing it to "self.config.get('vip', '').split() would return an empty list rather than breaking due to the None.

If there's no hacluster relation, then it should 'just work' (I think?) - i.e. it won't go into blocked and will just use it's normal ingress address, rather than a VIP.