Skip to content
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

DLPX-92722 recovery-environment fails to build on 24.04 due to erroneous path #44

Conversation

palash-gandhi
Copy link

@palash-gandhi palash-gandhi commented Nov 18, 2024

Pull request checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)
  • Build was run locally and any changes were pushed
  • Lint has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

recovery-environment failed to build on 24.04. Here's an example

16:41:57  + LD_LIBRARY_PATH='/tmp/tmp.cw5cz7ug8I/usr/lib/x86_64-linux-gnu/libfakeroot: (from /etc/ld.so.conf.d/fakeroot-x86_64-linux-gnu.conf:1)/tmp/tmp.cw5cz7ug8I/usr/local/lib: (from /etc/ld.so.conf.d/libc.conf:2)/tmp/tmp.cw5cz7ug8I/lib/x86_64-linux-gnu: (from /etc/ld.so.conf.d/x86_64-linux-gnu.conf:3)/tmp/tmp.cw5cz7ug8I/lib: (from <builtin>:0):/tmp/tmp.cw5cz7ug8I/lib/systemd'
16:41:57  + ldd /tmp/tmp.cw5cz7ug8I/img//bin/kmod
16:41:57  + echo 'linux-vdso.so.1 (0x00007ffcb678a000)'
16:41:57  + grep '=>'
16:41:57  + continue
16:41:57  + read -r line
16:41:57  + echo 'libzstd.so.1 => /lib/x86_64-linux-gnu/libzstd.so.1 (0x00007a7beee8b000)'
16:41:57  + grep '=>'
16:41:57  + local dep=
16:41:57  ++ echo 'libzstd.so.1 => /lib/x86_64-linux-gnu/libzstd.so.1 (0x00007a7beee8b000)'
16:41:57  ++ sed 's/.*=> \(.*\) (.*/\1/'
16:41:57  + dep=/lib/x86_64-linux-gnu/libzstd.so.1
16:41:57  + [[ /lib/x86_64-linux-gnu/libzstd.so.1 =~ /tmp/tmp.cw5cz7ug8I ]]
16:41:57  + die 'ldd found dependency outside of workdir'
16:41:57  + echo 'ldd found dependency outside of workdir'
16:41:57  ldd found dependency outside of workdir
16:41:57  + exit 1
16:41:57  make[1]: *** [Makefile:5: all] Error 1
16:41:57  make[1]: Leaving directory '/home/ubuntu/jenkins/workspace/linux-pkg/os-upgrade/build-package/recovery-environment/pre-push/linux-pkg/packages/recovery-environment/tmp/repo'
16:41:57  dh_auto_build: error: make -j2 returned exit code 2
16:41:57  make: *** [debian/rules:4: build] Error 25
16:41:57  dpkg-buildpackage: error: debian/rules build subprocess returned exit status 2
16:41:57  Error: failed command 'dpkg-buildpackage -b -us -uc'

Notice that the LDD_LIBRARY_PATH has those “from /etc/....” which is causing the path to be inferred incorrectly. Also, systemd-resolved was renamed to resolvectl and systemd-resolved now ships as a separate package beginning in 24.04: https://packages.ubuntu.com/search?keywords=systemd-resolved

What is the new behavior?

  • Fix LDD_LIBRARY_PATH such that it is inferred correctly.
  • Install systemd-resolved the same way the script installs systemd.

Does this introduce a breaking change?

  • Yes
  • No

Other information

build-package: https://ops-jenkins.eng-tools-prd.aws.delphixcloud.com/job/linux-pkg/job/os-upgrade/job/build-package/job/recovery-environment/job/pre-push/4/console

@palash-gandhi palash-gandhi force-pushed the dlpx/pr/palash-gandhi/d23faf19-998a-4d32-ab86-28465f1c07d1 branch 3 times, most recently from 24c864f to d9bf140 Compare November 18, 2024 18:45
@palash-gandhi palash-gandhi force-pushed the dlpx/pr/palash-gandhi/d23faf19-998a-4d32-ab86-28465f1c07d1 branch from d9bf140 to 2d80dcf Compare November 18, 2024 19:11
@palash-gandhi palash-gandhi marked this pull request as ready for review November 18, 2024 19:20
scripts/stage1.sh Outdated Show resolved Hide resolved
@palash-gandhi palash-gandhi merged commit 970f326 into os-upgrade Nov 21, 2024
2 of 7 checks passed
@palash-gandhi palash-gandhi deleted the dlpx/pr/palash-gandhi/d23faf19-998a-4d32-ab86-28465f1c07d1 branch November 21, 2024 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants