Skip to content

Conversation

tklauser
Copy link
Member

@tklauser tklauser commented Jun 3, 2025

Once this PR is merged, a GitHub action will update the labels of these PRs:

 37362 39802 39658

Artyop and others added 5 commits June 3, 2025 15:05
[ upstream commit db2feaf ]

This commit remove the usage of installing apt and curl inside the container by calling the metric endpoint from the runner instead.

Signed-off-by: Antony Reynaud <antony.reynaud@isovalent.com>
Signed-off-by: Tobias Klauser <tobias@cilium.io>
[ upstream commit 63be083 ]

The runners have enough disk-space to build the images, we can remove
this step which is slowing down CI build time.

Signed-off-by: André Martins <andre@cilium.io>
Signed-off-by: Tobias Klauser <tobias@cilium.io>
[ upstream commit db7ed53 ]

This reverts commit e90242e.

Signed-off-by: André Martins <andre@cilium.io>
Signed-off-by: Tobias Klauser <tobias@cilium.io>
[ upstream commit 56c985b ]

Remove temporary file in /mnt

In some runners, the file '/mnt/tmp-pv.img' is huge, taking the entire
space of '/mnt' directory. Thus, we will forcefully remove this file
which will allow us to re-use the /mnt directory again to build CI
images.

Signed-off-by: André Martins <andre@cilium.io>
Signed-off-by: Tobias Klauser <tobias@cilium.io>
[ upstream commit 3c5828b ]

Initially we checked for os.ErrInvalid, this is because it produces
the same string. However, os.ErrInvalid is defined in a OS independent
way in terms of an fs.ErrInvalid, so it does not actually match the
-EINVAL error that we want to check for.

Signed-off-by: Dylan Reimerink <dylan.reimerink@isovalent.com>
Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser tklauser added kind/backports This PR provides functionality previously merged into master. backport/1.17 This PR represents a backport for Cilium 1.17.x of a PR that was merged to main. labels Jun 3, 2025
@tklauser
Copy link
Member Author

tklauser commented Jun 3, 2025

/test

@tklauser tklauser marked this pull request as ready for review June 3, 2025 13:10
@tklauser tklauser requested review from a team as code owners June 3, 2025 13:10
Copy link
Contributor

@Artyop Artyop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good for 4d85c9c

@tklauser tklauser enabled auto-merge June 3, 2025 13:21
@tklauser tklauser added this pull request to the merge queue Jun 3, 2025
Merged via the queue into v1.17 with commit 31fb50b Jun 3, 2025
358 of 362 checks passed
@tklauser tklauser deleted the pr/v1.17-backport-2025-06-03-03-05 branch June 3, 2025 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.17 This PR represents a backport for Cilium 1.17.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants