Skip to content

Conversation

@mozzieongit
Copy link
Member

@mozzieongit mozzieongit commented Dec 3, 2025

  • Fix default shell in composite actions (for GitHub)
  • Fix inputs to composite action (was unnecessarily set to required, but is optional and needed a default)
  • Introduce a build profile environment variable (defaults to debug, can be overridden per job—theoretically even per step, but I think act has a bug there)
  • Add installed binaries (cascade{,d}/dnst) to PATH for easier use

Planned:

  • Introduce a settings script source file that can be used to fetch config file options/locations.
  • Document the default paths for stuff (Cascade, config, state, ...)
  • Disable release build until it is actually used by a test

Consider:

  • Replace bind9 with NSD serving .test and provide a script to update the DS RR for example.test? (I had used bind9 for it's DNS UPDATE capability)
  • Use a custom Docker image based on the slimmed down act image?
  • Check that error output from composite actions is visible when running the main workflow
  • Provide more errors in the test env management script when components fail (so lots of || { echo "bla"; exit 11; }; to make spotting failing components easier)
  • Maybe not try uploading /dev/null, but a file that is supported to avoid the warning ENTRYNOTSUPPORTED

@mozzieongit mozzieongit self-assigned this Dec 3, 2025
@mozzieongit
Copy link
Member Author

  • Fix default shell in composite actions (for GitHub)

This doesn't fix it actually, the defaults.run.shell option doesn't exist for composite actions. Instead I'll have to add the shell to each and every step

@ximon18
Copy link
Member

ximon18 commented Dec 12, 2025

Suggestion: don't tar the entire target directory, it's huge, only built binaries are actually wanted if I understand correctly.

@mozzieongit
Copy link
Member Author

Suggestion: don't tar the entire target directory, it's huge, only built binaries are actually wanted if I understand correctly.

Good idea

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants