Remove hardcodes, add in metadata
* Remove hardcodes of "Rocky" to use "shortname" instead * Add in metadata
This commit is contained in:
parent
4bc377cd44
commit
c1f1be9353
@ -15,7 +15,7 @@ What does this have?
|
|||||||
* iso -> ISO and Compose related utilities, primarily for Rocky Linux 9+
|
* iso -> ISO and Compose related utilities, primarily for Rocky Linux 9+
|
||||||
* live -> Live image related utilities
|
* live -> Live image related utilities
|
||||||
* mangle -> Manglers and other misc stuff
|
* mangle -> Manglers and other misc stuff
|
||||||
* sync -> Sync tools, primarily for Rocky Linux 8
|
* sync -> Sync tools, primarily for Rocky Linux 8 and will eventually be deprecated
|
||||||
|
|
||||||
How can I help?
|
How can I help?
|
||||||
---------------
|
---------------
|
||||||
@ -26,7 +26,7 @@ when you make changes:
|
|||||||
* Have pre-commit installed
|
* Have pre-commit installed
|
||||||
* Have shellcheck installed
|
* Have shellcheck installed
|
||||||
* Shell Scripts: These must pass a shellcheck test!
|
* Shell Scripts: These must pass a shellcheck test!
|
||||||
* Python scripts: Try your best to follow PEP8 guidelines
|
* Python scripts: Try your best to follow PEP8 guidelines (even the best linters get things wrong)
|
||||||
|
|
||||||
Your PR should be against the devel branch at all times. PR's against the main
|
Your PR should be against the devel branch at all times. PR's against the main
|
||||||
branch will be closed.
|
branch will be closed.
|
||||||
|
@ -12,6 +12,13 @@ cloud:
|
|||||||
- x86_64
|
- x86_64
|
||||||
project_id: ''
|
project_id: ''
|
||||||
'9':
|
'9':
|
||||||
|
cloud-kernel:
|
||||||
|
project_id: ''
|
||||||
|
allowed_arches:
|
||||||
|
- aarch64
|
||||||
|
- x86_64
|
||||||
|
- ppc64le
|
||||||
|
- s390x
|
||||||
cloud-common:
|
cloud-common:
|
||||||
project_id: ''
|
project_id: ''
|
||||||
allowed_arches:
|
allowed_arches:
|
||||||
|
@ -97,6 +97,12 @@ class RepoSync:
|
|||||||
self.extra_files = rlvars['extra_files']
|
self.extra_files = rlvars['extra_files']
|
||||||
self.gpgkey = gpgkey
|
self.gpgkey = gpgkey
|
||||||
|
|
||||||
|
self.compose_id = '{}-{}-{}'.format(
|
||||||
|
config['shortname'],
|
||||||
|
rlvars['revision'],
|
||||||
|
config['date_stamp']
|
||||||
|
)
|
||||||
|
|
||||||
# Templates
|
# Templates
|
||||||
file_loader = FileSystemLoader(f"{_rootdir}/templates")
|
file_loader = FileSystemLoader(f"{_rootdir}/templates")
|
||||||
self.tmplenv = Environment(loader=file_loader)
|
self.tmplenv = Environment(loader=file_loader)
|
||||||
@ -119,7 +125,10 @@ class RepoSync:
|
|||||||
self.compose_latest_dir = os.path.join(
|
self.compose_latest_dir = os.path.join(
|
||||||
config['compose_root'],
|
config['compose_root'],
|
||||||
major,
|
major,
|
||||||
"latest-Rocky-{}".format(self.profile)
|
"latest-{}-{}".format(
|
||||||
|
self.shortname,
|
||||||
|
self.profile
|
||||||
|
)
|
||||||
)
|
)
|
||||||
|
|
||||||
self.compose_latest_sync = os.path.join(
|
self.compose_latest_sync = os.path.join(
|
||||||
@ -224,7 +233,7 @@ class RepoSync:
|
|||||||
self.sync(self.repo, sync_root, work_root, log_root, global_work_root, self.arch)
|
self.sync(self.repo, sync_root, work_root, log_root, global_work_root, self.arch)
|
||||||
|
|
||||||
if self.fullrun:
|
if self.fullrun:
|
||||||
self.deploy_extra_files(global_work_root)
|
self.deploy_extra_files(sync_root, global_work_root)
|
||||||
self.deploy_treeinfo(self.repo, sync_root, self.arch)
|
self.deploy_treeinfo(self.repo, sync_root, self.arch)
|
||||||
self.symlink_to_latest(generated_dir)
|
self.symlink_to_latest(generated_dir)
|
||||||
|
|
||||||
@ -232,11 +241,13 @@ class RepoSync:
|
|||||||
self.repoclosure_work(sync_root, work_root, log_root)
|
self.repoclosure_work(sync_root, work_root, log_root)
|
||||||
|
|
||||||
if self.refresh_extra_files and not self.fullrun:
|
if self.refresh_extra_files and not self.fullrun:
|
||||||
self.deploy_extra_files(global_work_root)
|
self.deploy_extra_files(sync_root, global_work_root)
|
||||||
|
|
||||||
if self.refresh_treeinfo and not self.fullrun:
|
if self.refresh_treeinfo and not self.fullrun:
|
||||||
self.deploy_treeinfo(self.repo, sync_root, self.arch)
|
self.deploy_treeinfo(self.repo, sync_root, self.arch)
|
||||||
|
|
||||||
|
self.deploy_metadata(sync_root)
|
||||||
|
|
||||||
self.log.info('Compose repo directory: %s' % sync_root)
|
self.log.info('Compose repo directory: %s' % sync_root)
|
||||||
self.log.info('Compose logs: %s' % log_root)
|
self.log.info('Compose logs: %s' % log_root)
|
||||||
self.log.info('Compose completed.')
|
self.log.info('Compose completed.')
|
||||||
@ -576,7 +587,11 @@ class RepoSync:
|
|||||||
"""
|
"""
|
||||||
compose_base_dir = os.path.join(
|
compose_base_dir = os.path.join(
|
||||||
self.compose_base,
|
self.compose_base,
|
||||||
"Rocky-{}-{}".format(self.fullversion, self.date_stamp)
|
"{}-{}-{}".format(
|
||||||
|
self.shortname,
|
||||||
|
self.fullversion,
|
||||||
|
self.date_stamp
|
||||||
|
)
|
||||||
)
|
)
|
||||||
self.log.info('Creating compose directory %s' % compose_base_dir)
|
self.log.info('Creating compose directory %s' % compose_base_dir)
|
||||||
if not os.path.exists(compose_base_dir):
|
if not os.path.exists(compose_base_dir):
|
||||||
@ -872,7 +887,7 @@ class RepoSync:
|
|||||||
for issue in bad_exit_list:
|
for issue in bad_exit_list:
|
||||||
self.log.error(issue)
|
self.log.error(issue)
|
||||||
|
|
||||||
def deploy_extra_files(self, global_work_root):
|
def deploy_extra_files(self, sync_root, global_work_root):
|
||||||
"""
|
"""
|
||||||
deploys extra files based on info of rlvars including a
|
deploys extra files based on info of rlvars including a
|
||||||
extra_files.json
|
extra_files.json
|
||||||
@ -880,20 +895,27 @@ class RepoSync:
|
|||||||
might also deploy COMPOSE_ID and maybe in the future a metadata dir with
|
might also deploy COMPOSE_ID and maybe in the future a metadata dir with
|
||||||
a bunch of compose-esque stuff.
|
a bunch of compose-esque stuff.
|
||||||
"""
|
"""
|
||||||
|
self.log.info(
|
||||||
|
'[' + Color.BOLD + Color.GREEN + 'INFO' + Color.END + '] ' +
|
||||||
|
'Deploying treeinfo, discinfo, and media.repo'
|
||||||
|
)
|
||||||
|
|
||||||
cmd = self.git_cmd()
|
cmd = self.git_cmd()
|
||||||
tmpclone = '/tmp/clone'
|
tmpclone = '/tmp/clone'
|
||||||
extra_files_dir = os.path.join(
|
extra_files_dir = os.path.join(
|
||||||
global_work_root,
|
global_work_root,
|
||||||
'extra-files'
|
'extra-files'
|
||||||
)
|
)
|
||||||
self.log.info(
|
metadata_dir = os.path.join(
|
||||||
'[' + Color.BOLD + Color.GREEN + 'INFO' + Color.END + '] ' +
|
sync_root,
|
||||||
'Deploying extra files to work directory ...'
|
"metadata"
|
||||||
)
|
)
|
||||||
|
|
||||||
if not os.path.exists(extra_files_dir):
|
if not os.path.exists(extra_files_dir):
|
||||||
os.makedirs(extra_files_dir, exist_ok=True)
|
os.makedirs(extra_files_dir, exist_ok=True)
|
||||||
|
|
||||||
|
if not os.path.exists(metadata_dir):
|
||||||
|
os.makedirs(metadata_dir, exist_ok=True)
|
||||||
|
|
||||||
clonecmd = '{} clone {} -b {} -q {}'.format(
|
clonecmd = '{} clone {} -b {} -q {}'.format(
|
||||||
cmd,
|
cmd,
|
||||||
self.extra_files['git_repo'],
|
self.extra_files['git_repo'],
|
||||||
@ -907,6 +929,11 @@ class RepoSync:
|
|||||||
stderr=subprocess.DEVNULL
|
stderr=subprocess.DEVNULL
|
||||||
)
|
)
|
||||||
|
|
||||||
|
self.log.info(
|
||||||
|
'[' + Color.BOLD + Color.GREEN + 'INFO' + Color.END + '] ' +
|
||||||
|
'Deploying extra files to work and metadata directories ...'
|
||||||
|
)
|
||||||
|
|
||||||
# Copy files to work root
|
# Copy files to work root
|
||||||
for extra in self.extra_files['list']:
|
for extra in self.extra_files['list']:
|
||||||
src = '/tmp/clone/' + extra
|
src = '/tmp/clone/' + extra
|
||||||
@ -915,6 +942,7 @@ class RepoSync:
|
|||||||
# exist on our mirrors.
|
# exist on our mirrors.
|
||||||
try:
|
try:
|
||||||
shutil.copy2(src, extra_files_dir)
|
shutil.copy2(src, extra_files_dir)
|
||||||
|
shutil.copy2(src, metadata_dir)
|
||||||
except:
|
except:
|
||||||
self.log.warn(
|
self.log.warn(
|
||||||
'[' + Color.BOLD + Color.YELLOW + 'WARN' + Color.END + '] ' +
|
'[' + Color.BOLD + Color.YELLOW + 'WARN' + Color.END + '] ' +
|
||||||
@ -930,15 +958,48 @@ class RepoSync:
|
|||||||
e.strerror
|
e.strerror
|
||||||
)
|
)
|
||||||
|
|
||||||
|
def deploy_metadata(self, sync_root):
|
||||||
|
"""
|
||||||
|
Deploys metadata that defines information about the compose. Some data
|
||||||
|
will be close to how pungi produces it, but it won't be exact nor a
|
||||||
|
perfect replica.
|
||||||
|
"""
|
||||||
|
self.log.info(
|
||||||
|
'[' + Color.BOLD + Color.GREEN + 'INFO' + Color.END + '] ' +
|
||||||
|
'Deploying metadata for this compose'
|
||||||
|
)
|
||||||
# Create metadata here
|
# Create metadata here
|
||||||
# Create COMPOSE_ID here (this doesn't necessarily match anything, it's
|
# Create COMPOSE_ID here (this doesn't necessarily match anything, it's
|
||||||
# just an indicator)
|
# just an indicator)
|
||||||
|
metadata_dir = os.path.join(
|
||||||
|
sync_root,
|
||||||
|
"metadata"
|
||||||
|
)
|
||||||
|
|
||||||
|
# It should already exist from a full run or refresh. This is just in
|
||||||
|
# case and it doesn't hurt.
|
||||||
|
if not os.path.exists(metadata_dir):
|
||||||
|
os.makedirs(metadata_dir, exist_ok=True)
|
||||||
|
|
||||||
|
with open(metadata_dir + '/COMPOSE_ID') as f:
|
||||||
|
f.write(self.compose_id)
|
||||||
|
f.close()
|
||||||
|
|
||||||
|
Shared.write_metadata(
|
||||||
|
self.timestamp,
|
||||||
|
self.date_stamp,
|
||||||
|
self.fullname,
|
||||||
|
self.fullversion,
|
||||||
|
self.compose_id,
|
||||||
|
metadata_dir + 'metadata.json'
|
||||||
|
)
|
||||||
|
|
||||||
self.log.info(
|
self.log.info(
|
||||||
'[' + Color.BOLD + Color.GREEN + 'INFO' + Color.END + '] ' +
|
'[' + Color.BOLD + Color.GREEN + 'INFO' + Color.END + '] ' +
|
||||||
'Extra files phase completed.'
|
'Metadata files phase completed.'
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
def deploy_treeinfo(self, repo, sync_root, arch):
|
def deploy_treeinfo(self, repo, sync_root, arch):
|
||||||
"""
|
"""
|
||||||
Deploys initial treeinfo files. These have the potential of being
|
Deploys initial treeinfo files. These have the potential of being
|
||||||
@ -1181,6 +1242,7 @@ class SigRepoSync:
|
|||||||
major,
|
major,
|
||||||
repo=None,
|
repo=None,
|
||||||
arch=None,
|
arch=None,
|
||||||
|
ignore_debug: bool = False,
|
||||||
ignore_source: bool = False,
|
ignore_source: bool = False,
|
||||||
repoclosure: bool = False,
|
repoclosure: bool = False,
|
||||||
refresh_extra_files: bool = False,
|
refresh_extra_files: bool = False,
|
||||||
@ -1196,6 +1258,7 @@ class SigRepoSync:
|
|||||||
self.dryrun = dryrun
|
self.dryrun = dryrun
|
||||||
self.fullrun = fullrun
|
self.fullrun = fullrun
|
||||||
self.arch = arch
|
self.arch = arch
|
||||||
|
self.ignore_debug = ignore_debug
|
||||||
self.ignore_source = ignore_source
|
self.ignore_source = ignore_source
|
||||||
self.skip_all = skip_all
|
self.skip_all = skip_all
|
||||||
self.hashed = hashed
|
self.hashed = hashed
|
||||||
@ -1206,9 +1269,15 @@ class SigRepoSync:
|
|||||||
# Relevant config items
|
# Relevant config items
|
||||||
self.major_version = major
|
self.major_version = major
|
||||||
self.date_stamp = config['date_stamp']
|
self.date_stamp = config['date_stamp']
|
||||||
|
self.timestamp = time.time()
|
||||||
self.repo_base_url = config['repo_base_url']
|
self.repo_base_url = config['repo_base_url']
|
||||||
self.compose_root = config['compose_root']
|
self.compose_root = config['compose_root']
|
||||||
self.compose_base = config['compose_root'] + "/" + major
|
self.compose_base = config['compose_root'] + "/" + major
|
||||||
|
self.profile = rlvars['profile']
|
||||||
|
self.iso_map = rlvars['iso_map']
|
||||||
|
self.distname = config['distname']
|
||||||
|
self.fullname = rlvars['fullname']
|
||||||
|
self.shortname = config['shortname']
|
||||||
|
|
||||||
# Relevant major version items
|
# Relevant major version items
|
||||||
self.sigvars = sigvars
|
self.sigvars = sigvars
|
||||||
@ -1217,6 +1286,10 @@ class SigRepoSync:
|
|||||||
#self.project_id = sigvars['project_id']
|
#self.project_id = sigvars['project_id']
|
||||||
self.sigrepo = repo
|
self.sigrepo = repo
|
||||||
|
|
||||||
|
# Templates
|
||||||
|
file_loader = FileSystemLoader(f"{_rootdir}/templates")
|
||||||
|
self.tmplenv = Environment(loader=file_loader)
|
||||||
|
|
||||||
# each el can have its own designated container to run stuff in,
|
# each el can have its own designated container to run stuff in,
|
||||||
# otherwise we'll just default to the default config.
|
# otherwise we'll just default to the default config.
|
||||||
self.container = config['container']
|
self.container = config['container']
|
||||||
@ -1235,7 +1308,7 @@ class SigRepoSync:
|
|||||||
self.compose_latest_dir = os.path.join(
|
self.compose_latest_dir = os.path.join(
|
||||||
config['compose_root'],
|
config['compose_root'],
|
||||||
major,
|
major,
|
||||||
"latest-Rocky-{}-SIG".format(major)
|
"latest-{}-{}-SIG".format(self.shortname, major)
|
||||||
)
|
)
|
||||||
|
|
||||||
self.compose_latest_sync = os.path.join(
|
self.compose_latest_sync = os.path.join(
|
||||||
|
@ -135,7 +135,10 @@ class IsoBuild:
|
|||||||
self.compose_latest_dir = os.path.join(
|
self.compose_latest_dir = os.path.join(
|
||||||
config['compose_root'],
|
config['compose_root'],
|
||||||
major,
|
major,
|
||||||
"latest-Rocky-{}".format(self.profile)
|
"latest-{}-{}".format(
|
||||||
|
self.shortname,
|
||||||
|
self.profile
|
||||||
|
)
|
||||||
)
|
)
|
||||||
|
|
||||||
self.compose_latest_sync = os.path.join(
|
self.compose_latest_sync = os.path.join(
|
||||||
|
@ -1,6 +1,7 @@
|
|||||||
# These are shared utilities used
|
# These are shared utilities used
|
||||||
|
|
||||||
import os
|
import os
|
||||||
|
import json
|
||||||
import hashlib
|
import hashlib
|
||||||
import productmd.treeinfo
|
import productmd.treeinfo
|
||||||
|
|
||||||
@ -107,6 +108,37 @@ class Shared:
|
|||||||
the case of modifying treeinfo for primary repos or images.
|
the case of modifying treeinfo for primary repos or images.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
|
@staticmethod
|
||||||
|
def write_metadata(
|
||||||
|
timestamp,
|
||||||
|
datestamp,
|
||||||
|
fullname,
|
||||||
|
release,
|
||||||
|
compose_id,
|
||||||
|
file_path
|
||||||
|
):
|
||||||
|
|
||||||
|
metadata = {
|
||||||
|
"header": {
|
||||||
|
"name": "empanadas",
|
||||||
|
"version": "0.2.0",
|
||||||
|
"type": "toolkit",
|
||||||
|
"maintainer": "SIG/Core"
|
||||||
|
},
|
||||||
|
"payload": {
|
||||||
|
"compose": {
|
||||||
|
"date": datestamp,
|
||||||
|
"id": compose_id,
|
||||||
|
"fullname": fullname,
|
||||||
|
"release": release,
|
||||||
|
"timestamp": timestamp
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
with open(file_path, "w+") as f:
|
||||||
|
json.dump(metadata, f)
|
||||||
|
f.close()
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def discinfo_write(timestamp, fullname, arch, file_path):
|
def discinfo_write(timestamp, fullname, arch, file_path):
|
||||||
|
@ -22,6 +22,16 @@ if [ $ret_val -eq "0" ]; then
|
|||||||
sudo -l && find ** -maxdepth 0 -type l | parallel --will-cite -j 18 sudo rsync -av --chown=10004:10005 --progress --relative --human-readable \
|
sudo -l && find ** -maxdepth 0 -type l | parallel --will-cite -j 18 sudo rsync -av --chown=10004:10005 --progress --relative --human-readable \
|
||||||
{} "${TARGET}"
|
{} "${TARGET}"
|
||||||
|
|
||||||
|
# Temporary until empanadas has this support
|
||||||
|
if [ -f "COMPOSE_ID" ]; then
|
||||||
|
cp COMPOSE_ID "${TARGET}"
|
||||||
|
chown 10004:10005 "${TARGET}/COMPOSE_ID"
|
||||||
|
fi
|
||||||
|
|
||||||
|
if [ -d "metadata" ]; then
|
||||||
|
rsync -av --chown=10004:10005 --progress --relative --human-readable metadata "${TARGET}"
|
||||||
|
fi
|
||||||
|
|
||||||
# Full file list update
|
# Full file list update
|
||||||
cd "${PRODUCTION_ROOT}/${CATEGORY_STUB}/" || { echo "Failed to change directory"; exit 1; }
|
cd "${PRODUCTION_ROOT}/${CATEGORY_STUB}/" || { echo "Failed to change directory"; exit 1; }
|
||||||
# Hardlink everything except xml files
|
# Hardlink everything except xml files
|
||||||
|
@ -25,4 +25,14 @@ if [ $ret_val -eq "0" ]; then
|
|||||||
# shellcheck disable=SC2035
|
# shellcheck disable=SC2035
|
||||||
sudo -l && find **/* -maxdepth 0 -type d | parallel --will-cite -j 18 sudo rsync -av --chown=10004:10005 --progress --relative --human-readable \
|
sudo -l && find **/* -maxdepth 0 -type d | parallel --will-cite -j 18 sudo rsync -av --chown=10004:10005 --progress --relative --human-readable \
|
||||||
{} "${TARGET}"
|
{} "${TARGET}"
|
||||||
|
|
||||||
|
# This is temporary until we implement rsync into empanadas
|
||||||
|
if [ -f "COMPOSE_ID" ]; then
|
||||||
|
cp COMPOSE_ID "${TARGET}"
|
||||||
|
chown 10004:10005 "${TARGET}/COMPOSE_ID"
|
||||||
|
fi
|
||||||
|
|
||||||
|
if [ -d "metadata" ]; then
|
||||||
|
rsync -av --chown=10004:10005 --progress --relative --human-readable metadata "${TARGET}"
|
||||||
|
fi
|
||||||
fi
|
fi
|
||||||
|
Loading…
Reference in New Issue
Block a user