Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensuring changes to icon/webui labels are respected #10

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

arifer612
Copy link

This PR proposes 2 changes. The goal of these 2 changes are to resolve issue #9.

The first is to ensure that projects that are spun down are removed from
unRAID's cache by removing relevant entries in
/usr/local/emhttp/plugins/dynamix.docker.manager/docker.json. This ensures
that changes to the webui label are respected. This change is not a big change.

The second is to ensure that all cached icon files are removed when a project is
spun down. The files are cached in 2 locations: /var/lib/docker/unraid/images
and /usr/local/emhttp/plugins/dynamix.docker.manager/images. This is a hacky
attempt to ensure that changes to the icon label are respected. The reason I say
so is because this means icon files has to be downloaded every time the project
is spun up -- possibly many I/O operations.

Resolves #9

The changes in this commit clears any prior history of project services in
unRAID's cache. It ensures that changes made to the compose file gets properly
updated when the project is spun up.
The changes in this commit ensures that cached icons are removed from the server
so that changes made to the `net.unraid.docker.icon` are respected.
@arifer612 arifer612 force-pushed the feat/update-config branch from 8d884be to eb8a09c Compare June 24, 2022 11:09
@arifer612 arifer612 closed this Jun 25, 2022
@arifer612 arifer612 deleted the feat/update-config branch June 25, 2022 17:03
@arifer612 arifer612 restored the feat/update-config branch June 25, 2022 17:04
@arifer612
Copy link
Author

I did not realise that changing the branch name upstream would not reflect in PRs.

@arifer612 arifer612 reopened this Jun 25, 2022
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.

Changes to the the icon and webui labels not upheld
1 participant