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

Install nohang from source for ubuntu #2318

Merged
merged 4 commits into from
Mar 12, 2024
Merged

Conversation

liyimeng
Copy link
Contributor

@liyimeng liyimeng commented Mar 8, 2024

What this PR does / why we need it:

nohang apt repo is not actively maintained, install it from source instead.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #2195

@liyimeng
Copy link
Contributor Author

liyimeng commented Mar 8, 2024

@mauromorales make sense?

@liyimeng
Copy link
Contributor Author

liyimeng commented Mar 8, 2024

I manually build and test with ubuntu 22.04, it works. (it was failing early for me), but I have not test it on ubuntu 23 in particularly.

@mauromorales mauromorales requested a review from a team March 8, 2024 11:17
@mauromorales
Copy link
Member

@liyimeng thanks a lot for opening this PR

nohang apt repo is not actively maintained, install it from source instead.

try to fix kairos-io#2195

Signed-off-by: Liyi Meng <meng.mobile@gmail.com>
@mauromorales
Copy link
Member

@liyimeng I don't think nohang master has actually fixed the issue, they just don't panic on it but from the message, it seems to me that they still don't properly run when zfs is present

image

@liyimeng
Copy link
Contributor Author

@mauromorales you are right. I trace back the issue on GitHub. It dose not completely solve the problem since zfs is not providing a prober interface to calculate ARC accurately. However, it dose do some improvement.

1. We don't need zfs on UKI images
2. gnupg is not needed in common
3. Renaming of targets to legacy and current, to re-use between 22+
   versions

Signed-off-by: Mauro Morales <mauro.morales@spectrocloud.com>
Signed-off-by: Mauro Morales <mauro.morales@spectrocloud.com>
Signed-off-by: Mauro Morales <mauro.morales@spectrocloud.com>
@mauromorales
Copy link
Member

@liyimeng yup I agree. I did some refactoring. The first two are related to your changes and the last one just to make it easier to re-use the dockerfile.

  1. We don't need zfs on UKI images
  2. gnupg is not needed in common so keeping it on the nohang source target
  3. Renaming of targets to legacy and current, to re-use between 22+ versions which makes it more readable IMO

I will be merging once it's green.

Thanks a lot for your contribution and your feedback!

@mauromorales mauromorales merged commit e0d76a9 into kairos-io:master Mar 12, 2024
37 of 38 checks passed
@liyimeng liyimeng deleted the nohang branch March 13, 2024 05:25
@liyimeng
Copy link
Contributor Author

Thanks @mauromorales!

robarnold pushed a commit to marinatedconcrete/config that referenced this pull request May 20, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [kairos-io/kairos](https://togithub.com/kairos-io/kairos) | major |
`v2.5.0` -> `v3.0.11` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>kairos-io/kairos (kairos-io/kairos)</summary>

###
[`v3.0.11`](https://togithub.com/kairos-io/kairos/releases/tag/v3.0.11)

[Compare
Source](https://togithub.com/kairos-io/kairos/compare/v3.0.10...v3.0.11)

**Full Changelog**:
kairos-io/kairos@v3.0.10...v3.0.11

###
[`v3.0.10`](https://togithub.com/kairos-io/kairos/releases/tag/v3.0.10)

[Compare
Source](https://togithub.com/kairos-io/kairos/compare/v3.0.9...v3.0.10)

#### Updated

-   Bumps framework to v2.7.32
-   Bumps kairos-agent to v2.8.14 (part of the framework)

#### Fixes

- Prevent unwanted yaml fields to be marshalled: Fixes some step
duplication when triggering the install via events like providers do
- Avoid adding an extra line to os-release output: We were adding an
extra line at the end of the os-release file, which some software had
problems dealing with

#### Improvements

- `kairos-agent state` will now show SecureBoot state of the node
(on/off)
- `kairos-agent state` will now show the common name of the EFI
certificates in the node if any
- `kairos-agent state` will now show a list of encrypted partitions by
label and by device if any


![image](https://togithub.com/kairos-io/kairos/assets/1447686/d2059be4-1d9c-483e-9d89-0a6235e74e00)

**Full Changelog**:
kairos-io/kairos@v3.0.9...v3.0.10

###
[`v3.0.9`](https://togithub.com/kairos-io/kairos/releases/tag/v3.0.9)

[Compare
Source](https://togithub.com/kairos-io/kairos/compare/v3.0.8...v3.0.9)

- 🐛 Fix broken pipeline for arm rpi Tumbleweed (caused by Python
package being much bigger) by
[@&#8203;mauromorales](https://togithub.com/mauromorales)
[kairos-io/kairos#2524
- 🐛 Move nfs-utils to common build target in opensuse flavor by
[@&#8203;kaiehrhardt](https://togithub.com/kaiehrhardt)
[kairos-io/kairos#2495
- 🐛 UKI custom mounts breaking the cos-layout file by
[@&#8203;kreeuwijk](https://togithub.com/kreeuwijk)
[kairos-io/packages#839
(actual fix) release fix
([c95475c](https://togithub.com/kairos-io/kairos/pull/2524/commits/c95475c3e45c266c5cf9fc1ca2c96d446432dad4))
- ✨ Ubuntu 24.04 artifacts by
[@&#8203;mauromorales](https://togithub.com/mauromorales)
[#&#8203;2527](https://togithub.com/kairos-io/kairos/issues/2527)

**Full Changelog**:
kairos-io/kairos@v3.0.8...v3.0.9

###
[`v3.0.8`](https://togithub.com/kairos-io/kairos/releases/tag/v3.0.8)

[Compare
Source](https://togithub.com/kairos-io/kairos/compare/v3.0.7...v3.0.8)

Backported fixes for agent:

- Bumps yip for duplicated name of steps (was causing unexpected issues
with users sometimes) Issue:
[kairos-io/kairos#2488

#### Updated packages

|Category|Name|Old version|New Version|
|--|--|--|--|

|fips|kairos-agent|[v2.8.12](https://togithub.com/kairos-io/kairos-agent/releases/tag/v2.8.12)|[v2.8.13](https://togithub.com/kairos-io/kairos-agent/releases/tag/v2.8.13)|

|system|kairos-agent|[v2.8.12](https://togithub.com/kairos-io/kairos-agent/releases/tag/v2.8.12)|[v2.8.13](https://togithub.com/kairos-io/kairos-agent/releases/tag/v2.8.13)|

***

**Full Changelog**:
kairos-io/kairos@v3.0.7...v3.0.8

###
[`v3.0.7`](https://togithub.com/kairos-io/kairos/releases/tag/v3.0.7)

[Compare
Source](https://togithub.com/kairos-io/kairos/compare/v3.0.6...v3.0.7)

Backported fixes for agent and immucore:

-   Bumps yip for user id reuse
-   Adds sync calls before and after mounting
-   \[UKI] Fixes hooks in uki install
-   \[UKI] Fixes mounting oem before running rootfs stage under uki

#### Updated packages

|Category|Name|Old version|New Version|
|--|--|--|--|

|fips|kairos-agent|[v2.8.11](https://togithub.com/kairos-io/kairos-agent/releases/tag/v2.8.11)|[v2.8.12](https://togithub.com/kairos-io/kairos-agent/releases/tag/v2.8.12)|

|system|kairos-agent|[v2.8.11](https://togithub.com/kairos-io/kairos-agent/releases/tag/v2.8.11)|[v2.8.12](https://togithub.com/kairos-io/kairos-agent/releases/tag/v2.8.12)|

|system|immucore|[v0.1.25](https://togithub.com/kairos-io/immucore/releases/tag/v0.1.25)|[v0.1.26](https://togithub.com/kairos-io/immucore/releases/tag/v0.1.26)|

|fips|immucore|[v0.1.25](https://togithub.com/kairos-io/immucore/releases/tag/v0.1.25)|[v0.1.26](https://togithub.com/kairos-io/immucore/releases/tag/v0.1.26)|

***

**Full Changelog**:
kairos-io/kairos@v3.0.6...v3.0.7

###
[`v3.0.6`](https://togithub.com/kairos-io/kairos/releases/tag/v3.0.6)

[Compare
Source](https://togithub.com/kairos-io/kairos/compare/v3.0.5...v3.0.6)

#### What's Changed

- fix(orin): disable ISCSI in the initramfs generation by
[@&#8203;mudler](https://togithub.com/mudler) in
[kairos-io/kairos#2476

**Full Changelog**:
kairos-io/kairos@v3.0.5...v3.0.6

###
[`v3.0.5`](https://togithub.com/kairos-io/kairos/releases/tag/v3.0.5)

[Compare
Source](https://togithub.com/kairos-io/kairos/compare/v3.0.4...v3.0.5)

**Full Changelog**:
kairos-io/kairos@v3.0.4...v3.0.5

###
[`v3.0.4`](https://togithub.com/kairos-io/kairos/releases/tag/v3.0.4)

[Compare
Source](https://togithub.com/kairos-io/kairos/compare/v3.0.3...v3.0.4)

#### ⚠️ Known issues

🐛 Raspberry: EFI booting no longer supported on kernels shipped with
ubuntu > 22.04
[kairos-io/kairos#2249

We haven't been able to address the following issues on Alpine:
:bug: It's not possible to login on an Alpine 3.18 RPi
[#&#8203;2439](https://togithub.com/kairos-io/kairos/issues/2439)
🐛 Filesystem expansion on rpi4 doesn't work with Alpine
[kairos-io/kairos#1995
🐛 cgroup_memory not mounted in Alpine rpi4
[kairos-io/kairos#2002
🐛 Upgrade on alpine arm errors
[kairos-io/kairos#2135
🐛 reset from the GRUB menu on alpine, gets stuck in an endless loop
[kairos-io/kairos#2136

Deprecation warnings:

Reading of `/etc/elemental/config.yaml` is working again but will be
deprecated in favor of `/etc/kairos/config.yaml`

#### What's Changed

- Fix Reboot hangs for UKI images
[#&#8203;2384](https://togithub.com/kairos-io/kairos/issues/2384) by
[@&#8203;Itxaka](https://togithub.com/Itxaka)
- Remove snap from Ubuntu based images by
[@&#8203;mauromorales](https://togithub.com/mauromorales) in
[kairos-io/kairos#2415

**Full Changelog**:
kairos-io/kairos@v3.0.3...v3.0.4

###
[`v3.0.3`](https://togithub.com/kairos-io/kairos/releases/tag/v3.0.3)

[Compare
Source](https://togithub.com/kairos-io/kairos/compare/v3.0.2...v3.0.3)

#### ⚠️ Known issues

🐛 Raspberry: EFI booting no longer supported on kernels shipped with
ubuntu > 22.04
[kairos-io/kairos#2249

We haven't been able to address the following issues on Alpine:
:bug: It's not possible to login on an Alpine 3.18 RPi
[#&#8203;2439](https://togithub.com/kairos-io/kairos/issues/2439)
🐛 Filesystem expansion on rpi4 doesn't work with Alpine
[kairos-io/kairos#1995
🐛 cgroup_memory not mounted in Alpine rpi4
[kairos-io/kairos#2002
🐛 Upgrade on alpine arm errors
[kairos-io/kairos#2135
🐛 reset from the GRUB menu on alpine, gets stuck in an endless loop
[kairos-io/kairos#2136

Deprecation warnings:

Reading of `/etc/elemental/config.yaml` is working again but will be
deprecated in favor of `/etc/kairos/config.yaml`

#### What's Changed

- Fixes issue with the `bootentry` command in the agent when dealing
with UKI by [@&#8203;mauromorales](https://togithub.com/mauromorales) in
[kairos-io/kairos#2410

**Full Changelog**:
kairos-io/kairos@v3.0.2...v3.0.3

###
[`v3.0.2`](https://togithub.com/kairos-io/kairos/releases/tag/v3.0.2)

[Compare
Source](https://togithub.com/kairos-io/kairos/compare/v3.0.1...v3.0.2)

> \[!CAUTION]
> NOTE: The OpenSUSE Tumbleweed flavor of this release was affected by
the xz backdoor (https://nvd.nist.gov/vuln/detail/CVE-2024-3094). For
this reason all affected artifacts have been removed, both from quay.io
(container images) and from GitHub. If you used the Tumbleweed artifacts
of Kairos `3.0.1` or `3.0.2` you should immediately stop the affected
machines and re-install a version which is not affected. If those
systems were exposed to the internet (if ssh was possible) and they
included passwords or keys, it's advised that you rotate those
credentials.
>
> Read more:
>
> -   https://news.opensuse.org/2024/03/29/xz-backdoor/
> -
https://arstechnica.com/security/2024/04/what-we-know-about-the-xz-utils-backdoor-that-almost-infected-the-world/

#### ⚠️ Known issues

🐛 Raspberry: EFI booting no longer supported on kernels shipped with
ubuntu > 22.04
[kairos-io/kairos#2249

We haven't been able to address the following issues on Alpine:
:bug: It's not possible to login on an Alpine 3.18 RPi
[#&#8203;2439](https://togithub.com/kairos-io/kairos/issues/2439)
🐛 Filesystem expansion on rpi4 doesn't work with Alpine
[kairos-io/kairos#1995
🐛 cgroup_memory not mounted in Alpine rpi4
[kairos-io/kairos#2002
🐛 Upgrade on alpine arm errors
[kairos-io/kairos#2135
🐛 reset from the GRUB menu on alpine, gets stuck in an endless loop
[kairos-io/kairos#2136

Deprecation warnings:

Reading of `/etc/elemental/config.yaml` is working again but will be
deprecated in favor of `/etc/kairos/config.yaml`

#### What's Changed

- 🤖 Run fedora and ubuntu uki tests by
[@&#8203;Itxaka](https://togithub.com/Itxaka) in
[kairos-io/kairos#2366
- Replace a full test with a line in another test by
[@&#8203;jimmykarily](https://togithub.com/jimmykarily) in
[kairos-io/kairos#2368
- Reduce fedora uki image size by
[@&#8203;mauromorales](https://togithub.com/mauromorales) in
[kairos-io/kairos#2365
- chore(deps): update earthly/earthly docker tag to v0.8.6 by
[@&#8203;renovate](https://togithub.com/renovate) in
[kairos-io/kairos#2359
- 🤖 Allow manual testing of branches for uki jobs by
[@&#8203;Itxaka](https://togithub.com/Itxaka) in
[kairos-io/kairos#2373
- Do installation of kernel at the end of dockerfile by
[@&#8203;mauromorales](https://togithub.com/mauromorales) in
[kairos-io/kairos#2374
- chore(deps): update quay.io/kairos/osbuilder-tools docker tag to
v0.200.8 by [@&#8203;renovate](https://togithub.com/renovate) in
[kairos-io/kairos#2382
- fix(deps): update module github.com/kairos-io/kairos-sdk to v0.0.27 by
[@&#8203;renovate](https://togithub.com/renovate) in
[kairos-io/kairos#2383
- chore(deps): update dependency kairos-io/kairos-framework to v2.7.19
by [@&#8203;renovate](https://togithub.com/renovate) in
[kairos-io/kairos#2391
- Test that after-reset is run when in UKI mode by
[@&#8203;jimmykarily](https://togithub.com/jimmykarily) in
[kairos-io/kairos#2331
- 🤖 Bump osbuilder and drop keys by
[@&#8203;Itxaka](https://togithub.com/Itxaka) in
[kairos-io/kairos#2381
- fix(deps): update module github.com/kairos-io/kairos-sdk to v0.0.28 by
[@&#8203;renovate](https://togithub.com/renovate) in
[kairos-io/kairos#2394
- bump framework image by [@&#8203;mudler](https://togithub.com/mudler)
in
[kairos-io/kairos#2407

**Full Changelog**:
kairos-io/kairos@v3.0.1...v3.0.2

###
[`v3.0.1`](https://togithub.com/kairos-io/kairos/releases/tag/v3.0.1)

[Compare
Source](https://togithub.com/kairos-io/kairos/compare/v3.0.0...v3.0.1)

> \[!CAUTION]
> NOTE: The OpenSUSE Tumbleweed flavor of this release was affected by
the xz backdoor (https://nvd.nist.gov/vuln/detail/CVE-2024-3094). For
this reason all affected artifacts have been removed, both from quay.io
(container images) and from GitHub. If you used the Tumbleweed artifacts
of Kairos `3.0.1` or `3.0.2` you should immediately stop the affected
machines and re-install a version which is not affected. If those
systems were exposed to the internet (if ssh was possible) and they
included passwords or keys, it's advised that you rotate those
credentials.
>
> Read more:
>
> -   https://news.opensuse.org/2024/03/29/xz-backdoor/
> -
https://arstechnica.com/security/2024/04/what-we-know-about-the-xz-utils-backdoor-that-almost-infected-the-world/

#### ⚠️ Known issues

🐛 Raspberry: EFI booting no longer supported on kernels shipped with
ubuntu > 22.04
[kairos-io/kairos#2249

We haven't been able to address the following issues on Alpine:
:bug: It's not possible to login on an Alpine 3.18 RPi
[#&#8203;2439](https://togithub.com/kairos-io/kairos/issues/2439)
🐛 Filesystem expansion on rpi4 doesn't work with Alpine
[kairos-io/kairos#1995
🐛 cgroup_memory not mounted in Alpine rpi4
[kairos-io/kairos#2002
🐛 Upgrade on alpine arm errors
[kairos-io/kairos#2135
🐛 reset from the GRUB menu on alpine, gets stuck in an endless loop
[kairos-io/kairos#2136

Deprecation warnings:

Reading of `/etc/elemental/config.yaml` is working again but will be
deprecated in favor of `/etc/kairos/config.yaml`

#### What's Changed

- Add docs versioning item in the template by
[@&#8203;mauromorales](https://togithub.com/mauromorales) in
[kairos-io/kairos#2317
- Update earthly/earthly Docker tag to v0.8.5 by
[@&#8203;renovate](https://togithub.com/renovate) in
[kairos-io/kairos#2333
- Install nohang from source for ubuntu by
[@&#8203;liyimeng](https://togithub.com/liyimeng) in
[kairos-io/kairos#2318
- Update softprops/action-gh-release digest to
[`9d7c94c`](https://togithub.com/kairos-io/kairos/commit/9d7c94c) by
[@&#8203;renovate](https://togithub.com/renovate) in
[kairos-io/kairos#2320
- Update koalaman/shellcheck-alpine Docker tag to v0.10.0 by
[@&#8203;renovate](https://togithub.com/renovate) in
[kairos-io/kairos#2321
- Update quay.io/kairos/osbuilder-tools Docker tag to v0.200.6 -
autoclosed by [@&#8203;renovate](https://togithub.com/renovate) in
[kairos-io/kairos#2339
- Update dependency kairos-io/kairos-framework to v2.7.15 by
[@&#8203;renovate](https://togithub.com/renovate) in
[kairos-io/kairos#2344
- Move nfs-common to common build target in ubuntu flavor by
[@&#8203;kaiehrhardt](https://togithub.com/kaiehrhardt) in
[kairos-io/kairos#2340
- chore(deps): update quay.io/kairos/osbuilder-tools docker tag to
v0.200.7 by [@&#8203;renovate](https://togithub.com/renovate) in
[kairos-io/kairos#2350
- chore(deps): update quay.io/luet/base docker tag to v0.35.1 by
[@&#8203;renovate](https://togithub.com/renovate) in
[kairos-io/kairos#2352
- Bump framework to v2.7.17 by
[@&#8203;mauromorales](https://togithub.com/mauromorales) in
[kairos-io/kairos#2357

#### New Contributors

- [@&#8203;liyimeng](https://togithub.com/liyimeng) made their first
contribution in
[kairos-io/kairos#2318
- [@&#8203;kaiehrhardt](https://togithub.com/kaiehrhardt) made their
first contribution in
[kairos-io/kairos#2340

**Full Changelog**:
kairos-io/kairos@v3.0.0...v3.0.1

###
[`v3.0.0`](https://togithub.com/kairos-io/kairos/releases/tag/v3.0.0)

[Compare
Source](https://togithub.com/kairos-io/kairos/compare/v2.5.0...v3.0.0)

#### ⚠️ Known issues

🐛 Raspberry: EFI booting no longer supported on kernels shipped with
ubuntu > 22.04
[kairos-io/kairos#2249

We haven't been able to address the following issues on Alpine:
🐛 Filesystem expansion on rpi4 doesn't work with Alpine
[kairos-io/kairos#1995
🐛 cgroup_memory not mounted in Alpine rpi4
[kairos-io/kairos#2002
🐛 Upgrade on alpine arm errors
[kairos-io/kairos#2135
🐛 reset from the GRUB menu on alpine, gets stuck in an endless loop
[kairos-io/kairos#2136

Deprecation warnings:

Reading of `/etc/elemental/config.yaml` is working again but will be
deprecated in favor of `/etc/kairos/config.yaml`

#### New and noteworthy

- ✨ Enable secureboot for all flavors (minus alpine) by
[@&#8203;Itxaka](https://togithub.com/Itxaka) in
[kairos-io/kairos#2140
-   📖 Docs now have versioning starting with 3.0.0

#### What's Changed

- Add scp to fedora by
[@&#8203;jimmykarily](https://togithub.com/jimmykarily) in
[kairos-io/kairos#2154
- Remove duplicated zfs installation on rhel by
[@&#8203;mauromorales](https://togithub.com/mauromorales) in
[kairos-io/kairos#2161
- \[uki] Provide proper artifacts for auto-key-enrollment by
[@&#8203;Itxaka](https://togithub.com/Itxaka) in
[kairos-io/kairos#2172
- 🤖 Allow passing flags to enki and bump version by
[@&#8203;Itxaka](https://togithub.com/Itxaka) in
[kairos-io/kairos#2193
- 🐛 Do not recompress compressed firmware by
[@&#8203;Itxaka](https://togithub.com/Itxaka) in
[kairos-io/kairos#2237
- Have different compression depending on the ubuntu release by
[@&#8203;mauromorales](https://togithub.com/mauromorales) in
[kairos-io/kairos#2270
- Slim down Ubuntu by
[@&#8203;mauromorales](https://togithub.com/mauromorales) in
[kairos-io/kairos#2284
- Publish uki base image by
[@&#8203;mauromorales](https://togithub.com/mauromorales) in
[kairos-io/kairos#2290

**Full Changelog**:
kairos-io/kairos@v2.5.0...v3.0.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/marinatedconcrete/config).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yMzAuMCIsInVwZGF0ZWRJblZlciI6IjM3LjM2My41IiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

🐛 nohang service fails on Ubuntu 23.10
2 participants