Better detection of possible funkos

.ameba.yml

@@ -1,15 +1,16 @@
 # This configuration file was generated by `ameba --gen-config`
-# on 2024-07-02 16:33:24 UTC using Ameba version 1.6.1.
+# on 2024-07-02 22:36:34 UTC using Ameba version 1.6.1.
 # The point is for the user to remove these configuration records
 # one by one as the reported problems are removed from the code base.
 
-# Problems found: 11
+# Problems found: 12
 # Run `ameba --only Documentation/DocumentationAdmonition` for details
 Documentation/DocumentationAdmonition:
   Description: Reports documentation admonitions
   Timezone: UTC
   Excluded:
   - src/faaso.cr
+  - src/secrets.cr
   - src/daemon/main.cr
   - src/daemon/secrets.cr
   - src/funko.cr
@@ -29,7 +30,7 @@ Naming/BlockParameterName:
   AllowNamesEndingInNumbers: true
   Excluded:
   - src/faaso.cr
-  - src/daemon/funkos.cr
+  - src/daemon/funko.cr
   AllowedNames:
   - _
   - e

src/funko.cr

@@ -72,18 +72,25 @@ module Funko
     def self.from_docker : Array(Funko)
       docker_api = Docr::API.new(Docr::Client.new)
       names = [] of String
-      funko_containers = docker_api.containers.list(
+
+      # Get all containers that look like funkos
+      docker_api.containers.list(
         all: true,
       ).each { |container|
-        p! container.@names
         container.@names.each { |name|
           names << name.split("-", 2)[1].lstrip("/") if name.starts_with?("/faaso-")
         }
       }
 
-      pp! names
+      # Now get all images that look like funkos, since
-
+      # we can start them just fine.
-      from_names(names.to_a.sort!)
+      docker_api.images.list.each { |image|
+        next if image.@repo_tags.nil?
+        image.@repo_tags.as(Array(String)).each { |tag|
+          names << tag.split("-", 2)[1].split(":", 2)[0] if tag.starts_with?("faaso-")
+        }
+      }
+      from_names(names.to_set.to_a.sort!)
     end
 
     # Setup the target directory `path` with all the files needed

src/views/funkos.ecr

@@ -1,25 +1,32 @@
 <%- result.each do |f| -%>
 <tr>
   <td><%= f["name"] %></td>
-  <td><%= f["status"] %></td>
+  <%- if f["name"] == "proxy" -%>
+    <td>Running</td>
+  <%- else -%>
+    <td><%= f["status"] %></td>
+  <%- end -%>
   <td>
-  <%- if f["state"] == "running" -%>
+  <%- if f["name"] == "proxy" -%>
-    <button disabled>Start</button>
+  <%- else -%>
-    <button>Pause</button>
+    <%- if f["state"] == "running" -%>
-    <button>Stop</button>
+      <button disabled>Start</button>
-    <button>Restart</button>
+      <button>Pause</button>
-  <%- end -%>
+      <button>Stop</button>
-  <%- if f["state"] == "paused" -%>
+      <button>Restart</button>
-    <button>Start</button>
+    <%- end -%>
-    <button disabled>Pause</button>
+    <%- if f["state"] == "paused" -%>
-    <button>Stop</button>
+      <button>Start</button>
-    <button>Restart</button>
+      <button disabled>Pause</button>
-  <%- end -%>
+      <button>Stop</button>
-  <%- if f["state"] == "stopped" -%>
+      <button>Restart</button>
-    <button>Start</button>
+    <%- end -%>
-    <button disabled>Pause</button>
+    <%- if f["state"] == "stopped" -%>
-    <button disabled>Stop</button>
+      <button>Start</button>
-    <button disabled>Restart</button>
+      <button disabled>Pause</button>
+      <button disabled>Stop</button>
+      <button disabled>Restart</button>
+    <%- end -%>
   <%- end -%>
   </td>
 </tr>