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.

Sign up for GitHub

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

Jump to bottom

Clear marketplace cache during upgrade #1628

Merged
merged 1 commit into from
Jan 14, 2024
Merged

Clear marketplace cache during upgrade #1628

merged 1 commit into from
Jan 14, 2024

Conversation

J-N-K
Copy link
Member

@J-N-K J-N-K commented Jan 5, 2024

@J-N-K
Clear marketplace cache during upgrade
e321565 
Signed-off-by: Jan N. Klug <github@klug.nrw>
@J-N-K J-N-K added the awaiting other PR Depends on another PR label Jan 5, 2024
@J-N-K J-N-K requested a review from a team as a code owner January 5, 2024 17:47
@wborn wborn removed the awaiting other PR Depends on another PR label Jan 14, 2024
wborn
wborn approved these changes Jan 14, 2024
View reviewed changes
Copy link
Member

@wborn wborn left a comment

Choose a reason for hiding this comment

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

LGTM! I'll also add this to the Docker update scripts.

@wborn wborn merged commit cd4c3b7 into openhab:main Jan 14, 2024
3 checks passed
@wborn wborn added this to the 4.2 milestone Jan 14, 2024
@wborn wborn added the enhancement An enhancement or new feature label Jan 14, 2024
@jimtng
Copy link
Contributor

jimtng commented Mar 3, 2024

I wonder if this is causing my smarthome/j marketplace addon to disappear after every snapshot update (docker). It wasn't reinstalled either. I'm not sure where to start looking. As a workaround I moved the .kar file from marketplace/ to addons/.

@J-N-K J-N-K deleted the marketplace branch March 3, 2024 08:56
@J-N-K
Copy link
Member Author

J-N-K commented Mar 3, 2024

That's why they are removed, but they should be re-installed (and in fact that happens on my system). Can you try if you can re-produce the issue without using docker?

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

@wborn wborn wborn approved these changes

Assignees
No one assigned
Labels
enhancement An enhancement or new feature
Projects
None yet
Milestone
4.2
Development

Successfully merging this pull request may close these issues.
3 participants
@J-N-K @jimtng @wborn