On 3 October 2014 12:11, Raphaël Badin <email address hidden> wrote:
>
>
> Diff comments:
>
>> === modified file 'src/maasserver/enum.py'
>> --- src/maasserver/enum.py 2014-09-27 02:22:32 +0000
>> +++ src/maasserver/enum.py 2014-10-03 10:53:21 +0000
>> @@ -41,6 +41,7 @@
>> """Major moving parts of the application that may have failure states."""
>> PSERV = 'provisioning server'
>> IMPORT_PXE_FILES = 'maas-import-pxe-files script'
>> + CLUSTERS = 'clusters'
>
> We might want to have other component errors related to clusters. Maybe this should be renamed "DISCONNECTED_CLUSTERS"
Agreed.
>> class NODE_STATUS:
>>
>> + accepted_clusters = NodeGroup.objects.filter(
>> + status=NODEGROUP_STATUS.ACCEPTED)
>> + disconnected_clusters_found = any(
>> + not cluster.is_connected() for cluster in accepted_clusters)
>> + if disconnected_clusters_found:
>> + register_persistent_error(
>> + COMPONENT.CLUSTERS,
>> + "One or more clusters are currently disconnected. Visit "
>> + "the <a href=\"%s\">clusters</a> page for more "
>
> I think the link should apply to "clusters page" instead of just "clusters" but it's a detail.
>
On 3 October 2014 12:11, Raphaël Badin <email address hidden> wrote: /enum.py' enum.py 2014-09-27 02:22:32 +0000 enum.py 2014-10-03 10:53:21 +0000 pxe-files script' CLUSTERS"
>
>
> Diff comments:
>
>> === modified file 'src/maasserver
>> --- src/maasserver/
>> +++ src/maasserver/
>> @@ -41,6 +41,7 @@
>> """Major moving parts of the application that may have failure states."""
>> PSERV = 'provisioning server'
>> IMPORT_PXE_FILES = 'maas-import-
>> + CLUSTERS = 'clusters'
>
> We might want to have other component errors related to clusters. Maybe this should be renamed "DISCONNECTED_
Agreed.
>> class NODE_STATUS: objects. filter( NODEGROUP_ STATUS. ACCEPTED) clusters_ found = any( is_connected( ) for cluster in accepted_clusters) clusters_ found: persistent_ error( %s\">clusters< /a> page for more "
>>
>> + accepted_clusters = NodeGroup.
>> + status=
>> + disconnected_
>> + not cluster.
>> + if disconnected_
>> + register_
>> + COMPONENT.CLUSTERS,
>> + "One or more clusters are currently disconnected. Visit "
>> + "the <a href=\"
>
> I think the link should apply to "clusters page" instead of just "clusters" but it's a detail.
>
Fixed.