Skip to content

fix: multiple issues with local-api-data-watcher-pusher#4107

Open
rocketeerbkw wants to merge 1 commit into
mainfrom
fix-localdev-mcli
Open

fix: multiple issues with local-api-data-watcher-pusher#4107
rocketeerbkw wants to merge 1 commit into
mainfrom
fix-localdev-mcli

Conversation

@rocketeerbkw

@rocketeerbkw rocketeerbkw commented Jun 17, 2026

Copy link
Copy Markdown
Member

General Checklist

  • Affected Issues have been mentioned in the Closing issues section
  • Documentation has been written/updated
  • PR title is ready for inclusion in changelog

Database Migrations

n/a

Description

Fixes a number of issues discovered in local-api-data-watcher-pusher:

mcli: config is not a recognized command. Get help using --help flag.

There is no config command for mcli. Maybe it was removed in some version, but all the mcli cp calls have been failing silently for awhile. The files were being written to a folder called local-minio instead of the minio service. Fixed by using correct alias command.

$ git worktree remove test-mcli-error
warning: could not open directory 'local-dev/api-data-watcher-pusher/.mcli/': Permission denied
error: failed to delete 'test-mcli-error': Directory not empty

This is an annoyance for local development. Running mcli alias set creates a /home/.mcli folder with root permisisons. Since docker-compose mounts /home to the host it requires sudo to clean up the folder on the host. Fix is to run and mount scripts out of /app instead of /home.

[WARN tini (7)] Tini is not running as PID 1 and isn't registered as a child subreaper.
Zombie processes will not be re-parented to Tini, so zombie reaping won't work.
To fix the problem, use the -s option or set the environment variable TINI_SUBREAPER to register Tini as a child subreaper, or run Tini as PID 1.

Just a warning, but easy to fix by running tini as the entrypoint only, not in the command.

"errors":[{"message":"Error: Provided harbor configuration is not valid: Error: all branches, excluding pullrequests: latestPushed must be a number"

There is still one error left, will address in #4108.

Closing issues

n/a

 * `tini` not setup properly, throwing warnings
 * `mcli` using wrong command, silently failing to copy files
 * Dockerfile leaving runtime dirs on host, requiring `sudo` to cleanup
@shreddedbacon shreddedbacon self-requested a review June 18, 2026 02:43
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