sebb wrote on 1/24/25 6:39 PM:
On Fri, 24 Jan 2025 at 20:01, <curc...@apache.org> wrote:
...snip...
+    # @return hash, or string error if not found
      def self.get_timeline(mtg_root)
        begin
          return JSON.parse(IO.read(File.join(mtg_root, 'runbook', 
'timeline.json')))
        rescue StandardError => e
-        return null
+        return "ERROR: get_timeline(#{mtg_root}) threw: #{e.message}"

I don't see any checks on the return type by callers of this routine.
AFAICT they all assume it is a hash.

Correct - I still don't have a working dev environment so have been working on the server, which is fine since these tools are not yet actively being used. So I'm experimenting as I cleanup methods between the two nominate scripts, as well as adding the workflow features I'm proposing we use for the next go-round.

My concept was that if the caller cares, check the return value; if not a hash, then use the returned string as a specific error that provides additional debugging information.

Sadly, my --docker attempts fail on ldap, and my non-docker attempts fail with a "AH06665: No code signing... for mod_passenger.so" Which is odd, because homebrew says that they now do sign their modules...

Wondering if I'm just cursed with bad configs this week, or if this is something to do with Sequoia 15.2 on Apple Silicon M4 chip.

--
- Shane
  Member
  The Apache Software Foundation

Reply via email to