Bug 1298932

Description of problem:
The documentation in lvm2app.h specifies that lvm_vg_is_clustered() returns 1 if VG is clustered and 0 otherwise. However, the function returns 1024 if VG is clustered. This is because the macro vg_is_clustered in metadata-exported.h is set to

#define vg_is_clustered(vg) (vg_status((vg)) & CLUSTERED)

instead of

#define vg_is_clustered(vg) ((vg_status((vg)) & CLUSTERED) ? 1 : 0)

like in the case of for instance lv_is_locked

Other vg_is_* macros exhibit the similar issue.

The incorrect return value breaks the python wrapper as it does

rval = ( lvm_vg_is_clustered(self->vg) == 1) ? Py_True : Py_False;

which always returns Py_False regardless of the VG state.

How reproducible:
Always.

Steps to Reproduce:
Build and run thic C program
#include <lvm2app.h>
#include <stdio.h>

int main(int args, char**argv)
{
    lvm_t libh;
    vg_t vg = NULL;
    libh = lvm_init(NULL);
    vg = lvm_vg_open(libh, <clustered_vg_name>, "r", 0);
    printf("ret: %lu\n", lvm_vg_is_clustered(vg));
    lvm_vg_close(vg);
    lvm_quit(libh);
    return 0;
}

The program returns "ret: 1024".

Actual results:


Expected results:
lvm_vg_is_clustered() and others should return 1 if the specific flag is set in the VG.

Additional info: