Skip to content

Add Uptime kuma community container - #8495

Open
derStephan wants to merge 6 commits into
nextcloud:mainfrom
derStephan:uptime-Kuma-Container
Open

Add Uptime kuma community container#8495
derStephan wants to merge 6 commits into
nextcloud:mainfrom
derStephan:uptime-Kuma-Container

Conversation

@derStephan

@derStephan derStephan commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

This is my very first attempt for creating a community container. I will add another PR to integrate that into caddy.

Summary

  • The PR was tested and verified that it works locally
  • Sign-off message is added to all commits
  • Tests (playwright if possible) are included
  • Screenshots before/after for front-end changes
  • Documentation has been updated or is not required
  • Labels added where applicable (ex: bug/enhancement, 3. to review, feature component)
  • Milestone next added

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

derStephan added a commit to derStephan/aio-caddy that referenced this pull request Jul 29, 2026
see nextcloud/all-in-one#8495

Signed-off-by: derStephan <derStephan@users.noreply.github.com>
@szaimen szaimen added 2. developing Work in progress enhancement New feature or request labels Jul 31, 2026
@szaimen szaimen added this to the next milestone Jul 31, 2026
szaimen
szaimen previously requested changes Aug 3, 2026

@szaimen szaimen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, I am not really sure about this. See #7082 (reply in thread)

@szaimen szaimen removed this from the next milestone Aug 3, 2026
@derStephan

Copy link
Copy Markdown
Collaborator Author

I answered there.

@szaimen
szaimen dismissed their stale review August 4, 2026 13:47

Removing my comment so that @pablo can decide on this

@szaimen
szaimen requested a review from pabzm August 4, 2026 13:48
@szaimen szaimen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 4, 2026
@szaimen szaimen added this to the next milestone Aug 4, 2026
@derStephan derStephan closed this Aug 6, 2026
@derStephan
derStephan force-pushed the uptime-Kuma-Container branch from 2e734d9 to bbc9df8 Compare August 6, 2026 05:59
Signed-off-by: derStephan <derStephan@users.noreply.github.com>
Signed-off-by: derStephan <derStephan@users.noreply.github.com>
adds distinction to glances

Signed-off-by: derStephan <derStephan@users.noreply.github.com>
Signed-off-by: derStephan <derStephan@users.noreply.github.com>
@derStephan derStephan reopened this Aug 6, 2026
@szaimen szaimen modified the milestones: v13.5.0, next Aug 6, 2026
@pabzm

pabzm commented Aug 7, 2026

Copy link
Copy Markdown
Member

@derStephan Thank you for your first PR! 馃憦

I think it's a nice idea to integrate a full, standalone monitoring tool like this, and would prefer it over tools like "upptime" that rely on github.com.

My main problem with this is the integration of the docker socket, because it is such a gaping security boundary violation waiting to be exploited. I know some other community containers are using it, too, and I'm unhappy with that, and at least would like to not allow even more containers to use it.

From my perspective this would also be useful without the docker socket. It would require more configuration work, and provide less details automatically, but that's the price for better security protection, I'd say.

@derStephan Do you think you could change the integration to work like that? For Nextcloud there's e.g. /status.php as easy monitoring target.

Apart from this fundamental issue I'd have a few smaller remarks, which I'll hold back until we found a common ground.

@derStephan

Copy link
Copy Markdown
Collaborator Author

@pabzm Thank you for your thoughts.

I generally agree that this can be a security issue. However, it only allows read-only access to the socket. And without that, it is still useful for remote servers but not for the internal health documentation. Glances container is using this socket as well because without it, it will be useless.

The status.php requires a running nextcloud and will not alert in cases like #8437.

I can put a fat warning in the readme - as I did for the downtime during backups. Then the actual user can decide.

@derStephan

Copy link
Copy Markdown
Collaborator Author

done.

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

Labels

3. to review Waiting for reviews enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants