Skip to content

DLPX-94085 LTS 24.04: update delphix-platform for Ubuntu 24.04 appliance #527

New issue

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

By clicking “No 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? No Sign in to your account

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 9 additions & 13 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,12 @@ on: [push, pull_request]

jobs:
check-packages:
runs-on: ubuntu-latest
runs-on: ubuntu-24.04
steps:
- uses: actions/checkout@v2
- run: docker build -t delphix-platform:latest docker
- run: ./scripts/docker-run.sh make packages

check-shellcheck:
runs-on: ubuntu-latest
steps:
Expand All @@ -17,31 +18,26 @@ jobs:
steps:
- uses: actions/checkout@v2
- uses: delphix/actions/shfmt@master

check-pylint:
runs-on: ubuntu-latest
runs-on: ubuntu-24.04
steps:
- uses: actions/checkout@v2
- uses: actions/setup-python@v1
with:
python-version: '3.8'
- uses: actions/setup-python@v5
- run: python3 -m pip install pylint
- run: python3 -m pip install netifaces
- run: pylint -d invalid-name,E0611 files/common/usr/bin/delphix-startup-screen
check-yapf:
runs-on: ubuntu-latest
runs-on: ubuntu-24.04
steps:
- uses: actions/checkout@v2
- uses: actions/setup-python@v1
with:
python-version: '3.8'
- uses: actions/setup-python@v5
- run: python3 -m pip install yapf
- run: yapf --diff --style google files/common/usr/bin/delphix-startup-screen
check-mypy:
runs-on: ubuntu-latest
runs-on: ubuntu-24.04
steps:
- uses: actions/checkout@v2
- uses: actions/setup-python@v1
with:
python-version: '3.8'
- uses: actions/setup-python@v5
- run: python3 -m pip install mypy
- run: mypy --ignore-missing-imports files/common/usr/bin/delphix-startup-screen
18 changes: 15 additions & 3 deletions debian/postinst
Original file line number Diff line number Diff line change
Expand Up @@ -52,13 +52,23 @@ configure)
#
systemctl disable nullmailer.service

systemctl unmask auditd.service
systemctl enable auditd.service
systemctl enable delphix.target
systemctl enable delphix-platform.service
systemctl enable delphix-rpool-upgrade.service

systemctl unmask systemd-networkd.service
systemctl enable systemd-networkd.service

systemctl unmask iscsi-name-init.service
systemctl enable iscsi-name-init.service

systemctl unmask delphix-platform.service
systemctl unmask delphix-rpool-upgrade.service
systemctl unmask delphix.target

systemctl enable delphix-platform.service
systemctl enable delphix-rpool-upgrade.service
systemctl enable delphix.target

if ! id -u postgres >/dev/null; then
# When installing postgres, a postgres user is created unless it
# already exists. To have a consistent UID accross installations
Expand All @@ -70,6 +80,8 @@ configure)
# postgres UID. Note that we put this code here instead of the
# delphix-platform service as we need it to be executed before
# the postgres package gets installed.
sed -i 's/^UID_MAX.*/UID_MAX 65437/g' /etc/.defs
sed -i 's/^GID_MAX.*/GID_MAX 65437/g' /etc/.defs
addgroup postgres --gid 65437
adduser --home /var/lib/postgresql --no-create-home \
--shell /bin/bash --ingroup postgres \
Expand Down
31 changes: 6 additions & 25 deletions debian/rules
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/usr/bin/make -f
#
# Copyright 2018, 2024 Delphix
# Copyright 2018, 2025 Delphix
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -72,19 +72,21 @@ DEPENDS += ansible, \
net-tools, \
netbase, \
netplan.io, \
ntp, \
nullmailer, \
ntpsec, \
nullmailer, \
open-iscsi, \
openssh-server, \
openssl, \
passwd, \
policykit-1, \
procps, \
python3, \
python3-netifaces, \
rng-tools, \
rsyslog, \
sudo, \
systemd-container, \
systemd-resolved, \
tzdata, \
udev,

Expand All @@ -110,47 +112,27 @@ DEPENDS += $(DEPENDS.$(TARGET_PLATFORM))
#
DEPENDS += delphix-build-info,

#
# The usrmerge package modifies the layout of directories under root (/) upon
# installation, to ensure that a Delphix Engine upgraded to Ubuntu 20.04 has
# the same directory layout as a Delphix Engine that initially came on
# Ubuntu 20.04 (or later).
#
DEPENDS += usrmerge,

#
# These packages help strengthen the security of the appliance by identifying
# and preventing undesired behaviors.
#
DEPENDS += aide, \
aide-common,

#
# These packages are tools that are intended for human convenience. The
# product should not rely on them programmatically. They may be updated
# or replaced without regard for backward compatibility.
#
DEPENDS += aptitude, \
awscli, \
bcc-tools, \
bcc, \
bpftrace, \
crash-python, \
delphix-rust, \
dnsutils, \
drgn, \
dstat, \
emacs-nox, \
ethtool, \
gdb, \
gdb-python, \
htop, \
iftop, \
inotify-tools, \
iotop, \
jq, \
kdump-tools, \
ldap-utils, \
libkdumpfile, \
linux-tools-common, \
lsof, \
man-db, \
Expand Down Expand Up @@ -229,4 +211,3 @@ override_dh_auto_test:
# testing via another mechanism, so running it during package
# builds is unnecessary.
#

10 changes: 0 additions & 10 deletions files/common/lib/modprobe.d/10-lockd.conf

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
when: ansible_virtualization_type != "systemd-nspawn" and not ansible_is_chroot

- systemd:
name: sshd
name: ssh
state: reloaded
listen: "sshd config changed"
when: ansible_virtualization_type != "systemd-nspawn" and not ansible_is_chroot
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -349,7 +349,7 @@
regexp: '^{{ item.key }}='
line: '{{ item.key }}="{{ item.value }}"'
with_items:
- { key: 'JAVA_HOME', value: '/usr/lib/jvm/adoptopenjdk-java8-jdk-amd64' }
- { key: 'JAVA_HOME', value: '/usr/lib/jvm/java-8-openjdk-amd64' }

#
# Configure the Azure agent. Only run this on Azure, since that is the
Expand Down
Loading