From 2ba7c8f0a5c5dbbd62a1f7ab58a87891265f6c22 Mon Sep 17 00:00:00 2001 From: Yihuang Yu Date: Mon, 29 Apr 2024 21:22:16 +0800 Subject: [PATCH] refactor: udpate DISK_USER and DISK_GROUP DISK_USER and DISK_GROUP are from #299, they are used for bib option `--chown`. However, the current variable name may cause misunderstanding, mislead people into thinking it's a user of operation system inside disk, so rename them. Also, add these 2 variables into the README. Signed-off-by: Yihuang Yu --- recipes/common/Makefile.common | 6 +++--- recipes/common/README.md | 2 ++ 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/recipes/common/Makefile.common b/recipes/common/Makefile.common index 2efabf5c..6a3b13e3 100644 --- a/recipes/common/Makefile.common +++ b/recipes/common/Makefile.common @@ -10,8 +10,8 @@ SSH_PUBKEY ?= $(shell cat ${HOME}/.ssh/id_rsa.pub;) BOOTC_IMAGE ?= quay.io/$(REGISTRY_ORG)/${APP}-bootc:latest BOOTC_IMAGE_BUILDER ?= quay.io/centos-bootc/bootc-image-builder DISK_TYPE ?= qcow2 -DISK_USER ?= $(shell id -u) -DISK_GROUP ?= $(shell id -g) +DISK_UID ?= $(shell id -u) +DISK_GID ?= $(shell id -g) FROM ?= ARCH ?= CONTAINERFILE ?= Containerfile @@ -127,7 +127,7 @@ bootc-image-builder: $(BOOTC_IMAGE_BUILDER) \ $(ARCH:%=--target-arch %) \ --type $(DISK_TYPE) \ - --chown $(DISK_USER):$(DISK_GROUP) \ + --chown $(DISK_UID):$(DISK_GID) \ --local \ $(BOOTC_IMAGE) diff --git a/recipes/common/README.md b/recipes/common/README.md index 2316f698..3d0fc9e9 100644 --- a/recipes/common/README.md +++ b/recipes/common/README.md @@ -28,6 +28,8 @@ used to override defaults for a variety of make targets. |BOOTC_IMAGE_BUILDER | Bootc Image Builder container image | `quay.io/centos-bootc/bootc-image-builder` | |CHROMADB_IMAGE | ChromaDB image to be used for application | `$(REGISTRY)/$(REGISTRY_ORG)/chromadb:latest` | |DISK_TYPE | Disk type to be created by BOOTC_IMAGE_BUILDER | `qcow2` (Options: ami, iso, vmdk, raw) | +|DISK_UID | Disk UID to be specified by BOOTC_IMAGE_BUILDER | `$(shell id -u)` | +|DISK_GID | Disk GID to be specified by BOOTC_IMAGE_BUILDER | `$(shell id -g)` | |MODEL_IMAGE | AI Model to be used by application | `$(REGISTRY)/$(REGISTRY_ORG)/mistral-7b-instruct:latest`| |SERVER_IMAGE | AI Model Server Application | `$(REGISTRY)/$(REGISTRY_ORG)/llamacpp_python:latest` | |SSH_PUBKEY | SSH Public key preloaded in bootc image. | `$(shell cat ${HOME}/.ssh/id_rsa.pub;)` |