-
-
Notifications
You must be signed in to change notification settings - Fork 380
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feature: Add an admin notice about the usage of the legacy container …
…images
- Loading branch information
1 parent
2245887
commit 4bfb3b4
Showing
7 changed files
with
229 additions
and
9 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
export function AdminCard({ children }: { children: React.ReactNode }) { | ||
return <div className="rounded-md border bg-background p-4">{children}</div>; | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,71 @@ | ||
"use client"; | ||
|
||
import { Alert, AlertDescription, AlertTitle } from "@/components/ui/alert"; | ||
import { Badge } from "@/components/ui/badge"; | ||
import { api } from "@/lib/trpc"; | ||
import { AlertCircle } from "lucide-react"; | ||
|
||
import { AdminCard } from "./AdminCard"; | ||
|
||
interface AdminNotice { | ||
level: "info" | "warning" | "error"; | ||
message: React.ReactNode; | ||
title: string; | ||
} | ||
|
||
function useAdminNotices() { | ||
const { data } = api.admin.getAdminNoticies.useQuery(); | ||
if (!data) { | ||
return []; | ||
} | ||
const ret: AdminNotice[] = []; | ||
if (data.legacyContainersNotice) { | ||
ret.push({ | ||
level: "warning", | ||
message: ( | ||
<p> | ||
You're using the legacy docker container images. Those will stop | ||
getting supported soon. Please follow{" "} | ||
<a | ||
href="https://docs.hoarder.app/next/Guides/legacy-container-upgrade" | ||
className="underline" | ||
> | ||
this guide | ||
</a>{" "} | ||
to upgrade. | ||
</p> | ||
), | ||
title: "Legacy Container Images", | ||
}); | ||
} | ||
return ret; | ||
} | ||
|
||
export function AdminNotices() { | ||
const notices = useAdminNotices(); | ||
|
||
if (notices.length === 0) { | ||
return null; | ||
} | ||
return ( | ||
<AdminCard> | ||
<div className="flex flex-col gap-2"> | ||
{notices.map((n, i) => ( | ||
<Alert key={i} variant="destructive"> | ||
<AlertCircle className="h-4 w-4" /> | ||
<AlertTitle>{n.title}</AlertTitle> | ||
<AlertDescription>{n.message}</AlertDescription> | ||
</Alert> | ||
))} | ||
</div> | ||
</AdminCard> | ||
); | ||
} | ||
|
||
export function AdminNoticeBadge() { | ||
const notices = useAdminNotices(); | ||
if (notices.length === 0) { | ||
return null; | ||
} | ||
return <Badge variant="destructive">{notices.length}</Badge>; | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,60 @@ | ||
import type { VariantProps } from "class-variance-authority"; | ||
import * as React from "react"; | ||
import { cn } from "@/lib/utils"; | ||
import { cva } from "class-variance-authority"; | ||
|
||
const alertVariants = cva( | ||
"relative w-full rounded-lg border p-4 [&>svg+div]:translate-y-[-3px] [&>svg]:absolute [&>svg]:left-4 [&>svg]:top-4 [&>svg]:text-foreground [&>svg~*]:pl-7", | ||
{ | ||
variants: { | ||
variant: { | ||
default: "bg-background text-foreground", | ||
destructive: | ||
"border-destructive/50 text-destructive dark:border-destructive [&>svg]:text-destructive", | ||
}, | ||
}, | ||
defaultVariants: { | ||
variant: "default", | ||
}, | ||
}, | ||
); | ||
|
||
const Alert = React.forwardRef< | ||
HTMLDivElement, | ||
React.HTMLAttributes<HTMLDivElement> & VariantProps<typeof alertVariants> | ||
>(({ className, variant, ...props }, ref) => ( | ||
<div | ||
ref={ref} | ||
role="alert" | ||
className={cn(alertVariants({ variant }), className)} | ||
{...props} | ||
/> | ||
)); | ||
Alert.displayName = "Alert"; | ||
|
||
const AlertTitle = React.forwardRef< | ||
HTMLParagraphElement, | ||
React.HTMLAttributes<HTMLHeadingElement> | ||
>(({ className, ...props }, ref) => ( | ||
// eslint-disable-next-line jsx-a11y/heading-has-content | ||
<h5 | ||
ref={ref} | ||
className={cn("mb-1 font-medium leading-none tracking-tight", className)} | ||
{...props} | ||
/> | ||
)); | ||
AlertTitle.displayName = "AlertTitle"; | ||
|
||
const AlertDescription = React.forwardRef< | ||
HTMLParagraphElement, | ||
React.HTMLAttributes<HTMLParagraphElement> | ||
>(({ className, ...props }, ref) => ( | ||
<div | ||
ref={ref} | ||
className={cn("text-sm [&_p]:leading-relaxed", className)} | ||
{...props} | ||
/> | ||
)); | ||
AlertDescription.displayName = "AlertDescription"; | ||
|
||
export { Alert, AlertTitle, AlertDescription }; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,66 @@ | ||
# Legacy Container Upgrade | ||
|
||
Hoarder's 0.16 release consolidated the web and worker containers into a single container and also dropped the need for the redis container. The legacy containers will stop being supported soon, to upgrade to the new container do the following: | ||
|
||
1. Remove the redis container and its volume if it had one. | ||
2. Move the environment variables that you've set exclusively to the `workers` container to the `web` container. | ||
3. Delete the `workers` container. | ||
4. Rename the web container image from `hoarder-app/hoarder-web` to `hoarder-app/hoarder`. | ||
|
||
```diff | ||
diff --git a/docker/docker-compose.yml b/docker/docker-compose.yml | ||
index cdfc908..6297563 100644 | ||
--- a/docker/docker-compose.yml | ||
+++ b/docker/docker-compose.yml | ||
@@ -1,7 +1,7 @@ | ||
version: "3.8" | ||
services: | ||
web: | ||
- image: ghcr.io/hoarder-app/hoarder-web:${HOARDER_VERSION:-release} | ||
+ image: ghcr.io/hoarder-app/hoarder:${HOARDER_VERSION:-release} | ||
restart: unless-stopped | ||
volumes: | ||
- data:/data | ||
@@ -10,14 +10,10 @@ services: | ||
env_file: | ||
- .env | ||
environment: | ||
- REDIS_HOST: redis | ||
MEILI_ADDR: http://meilisearch:7700 | ||
+ BROWSER_WEB_URL: http://chrome:9222 | ||
+ # OPENAI_API_KEY: ... | ||
DATA_DIR: /data | ||
- redis: | ||
- image: redis:7.2-alpine | ||
- restart: unless-stopped | ||
- volumes: | ||
- - redis:/data | ||
chrome: | ||
image: gcr.io/zenika-hub/alpine-chrome:123 | ||
restart: unless-stopped | ||
@@ -37,24 +33,7 @@ services: | ||
MEILI_NO_ANALYTICS: "true" | ||
volumes: | ||
- meilisearch:/meili_data | ||
- workers: | ||
- image: ghcr.io/hoarder-app/hoarder-workers:${HOARDER_VERSION:-release} | ||
- restart: unless-stopped | ||
- volumes: | ||
- - data:/data | ||
- env_file: | ||
- - .env | ||
- environment: | ||
- REDIS_HOST: redis | ||
- MEILI_ADDR: http://meilisearch:7700 | ||
- BROWSER_WEB_URL: http://chrome:9222 | ||
- DATA_DIR: /data | ||
- # OPENAI_API_KEY: ... | ||
- depends_on: | ||
- web: | ||
- condition: service_started | ||
|
||
volumes: | ||
- redis: | ||
meilisearch: | ||
data: | ||
``` |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters