-
Notifications
You must be signed in to change notification settings - Fork 44
DLPX-89763 DLPX-86523 delphix-platform changes #477
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.
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
base: develop
Are you sure you want to change the base?
DLPX-89763 DLPX-86523 delphix-platform changes #477
Conversation
ec9dada
to
317e58e
Compare
...n/var/lib/delphix-platform/ansible/10-delphix-platform/roles/delphix-platform/tasks/main.yml
Show resolved
Hide resolved
317e58e
to
68118d0
Compare
68118d0
to
e56b7de
Compare
a4f876c
to
d356ef1
Compare
38fba97
to
28cab6b
Compare
b1d61e7
to
0c718f3
Compare
b984a71
to
f389cbe
Compare
64091f7
to
5022542
Compare
5022542
to
744dc46
Compare
d39c466
to
52eee14
Compare
b5743b5
to
3a9629e
Compare
4e5c764
to
d0ba9f9
Compare
d0ba9f9
to
43d194d
Compare
...n/var/lib/delphix-platform/ansible/10-delphix-platform/roles/delphix-platform/tasks/main.yml
Outdated
Show resolved
Hide resolved
24f6877
to
5a17a79
Compare
We have made the latest changes here and added the below logic to
We this change we got a build : appliance-build-orchestrator-pre-push/10842.
-- Created an app data unstructured vdb on a target Also my /var/log/syslog show
Upon executing below
And my log shows that the services has started successfully
Setup Present for the same setup: dlpx-qa-2025200-qar-170885-766f97e2.dlpxdc.co |
Now here, we have created an alternative PR where we have added the logic to cleanup the We have added another service delphix-legacy-link.service This Build is Validated : https://selfservice-jenkins.eng-tools-prd.aws.delphixcloud.com/job/appliance-build-orchestrator-pre-push/10846/ |
Post changing my code as per above comments, I have Build appliance-build-orchestrator-pre-push/10897 running. |
27a3f5e
to
8f0067f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we're almost there with this. The only remaining detail is the mechanism used to run one-time boot-time logic.
f2356e1
to
e003914
Compare
1c3baff
to
e22f588
Compare
e22f588
to
26210f6
Compare
8b5f0fb
to
a5ef067
Compare
debian/postinst
Outdated
if [[ -f "/var/lib/delphix-platform/export-home" ]]; then | ||
chmod 755 "/var/lib/delphix-platform/export-home" | ||
fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This shouldn't be necessary. Note that there is no equivalent code needed for the other executables delivered by this package. You should be able to set the permissions on the file directly in the repo.
echo 'Ensuring /export/home is a symlink to /home...' | ||
if [ -e /export/home ]; then | ||
echo 'Removing existing /export/home directory...' | ||
rm -rf /export/home |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems risky. If there's some bug in our code and we fail to unmount home directories for whatever reason, this will delete all home directories. At this point, if everything works as designed, /export/home
should be an empty directory (it's the mountpoint for a filesystem that should have been unmounted already), and so rmdir
should succeed.
…rt/home Adding the mount point logic back to delphix-platform.sh script Changing as per Seb-s comment to alter the softlink logic. Changing the code to previous version to validate the comments. Changing the delphix-legacy-link.service to perform the same steps next reboot like deferred upgrade. Creating a different service for taking care of soft /export/home -> /home DLPX-89763 DLPX-86523 delphix-platform changes Moved the code to create symlink to different script Removing the autofs handling as per the concerns raised by Seb and our validation Removing the softlink loginc from main.yml PR URL: https://www.github.com/delphix/delphix-platform/pull/477
eba0198
to
b36fe93
Compare
Problem
CIS is looking or a single home directory filesystem mounted at the
/home
location, currently we have the home dataset is mounted on/export/home
Due to that we see the below issues in the CIS Report
Solution
/home
./export/home
to/home
so that the exiting tests and dependencies have minimal impact.We have now implemented the
delphix-legacy-link.service
for the/export/home -> /home
creation, we earlier tried to do the sae in thedelphix-platform.services
and tested to implement the change withExecStart
andExecStartPost
. It dod not work and then this change was implemented to achieve the same after thedelphix-platform.services
are up completely.Testing Done
-------Start of Testing of Build 11635 13 July 2025 ----------
Git ab-pre-push : 11635 - 🟠
Pre-checkin: Initial pre-checking job with above was failed and later passed with https://github.com/delphix/qa-infra/pull/2315
Successful Pre-chekin: https://selfservice-jenkins.eng-tools-prd.aws.delphixcloud.com/job/blackbox-self-service/183042/consoleFull - 🟢
Additional Automated Upgrade 2025.20.0 --> develop : https://selfservice-jenkins.eng-tools-prd.aws.delphixcloud.com/job/blackbox-chained/7870/consoleFull - 🟢
Manual Upgrade upgrade from dlpx-2025.3.0.0 --> develop : 🟢 | Engine : dlpx2025300.dlpxdc.co | Alive : 🟢

In manual upgrade we created an appdata dsource and created a vdb on the target engine, after upgrade our dsource was Active & the vdb was founf in Running state.
Scan Results
Created a Engine from the AMI published from the appliance job above
As we are now upgraded to Ubuntu 24.04 , there are changes in the CIS test, we have Ran the CIS Scans with the now with a new policy. we have some changes 7402 , 7403, 14601 and 13248 are not present in the new policy instead below 3 new controls closer to
CIS Scan Report
lsof output, no dependency on /export/home -
Softlink /export/home -> /home - 🟢
-------End of Testing of Build 11635 13 July 2025----------