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

Make sure /etc/mender/scripts/version is closed after use. #916

Merged
merged 1 commit into from Jan 11, 2022

Conversation

kacf
Copy link
Member

@kacf kacf commented Jan 10, 2022

Changelog: Fix a (possible) file descriptor leak.

Signed-off-by: Kristian Amlie kristian.amlie@northern.tech

@mender-test-bot
Copy link

@kacf, Let me know if you want to start the integration pipeline by mentioning me and the command "start pipeline".


my commands and options
You can trigger a pipeline on multiple prs with:
  • mentioning me and start pipeline --pr mender/127 --pr mender-connect/255

You can cherry pick to a given branch or branches with:

  • mentioning me and:
 cherry-pick to:
 * 1.0.x
 * 2.0.x

Changelog: Fix a (possible) file descriptor leak.

Signed-off-by: Kristian Amlie <kristian.amlie@northern.tech>
@kacf
Copy link
Member Author

kacf commented Jan 10, 2022

@mender-test-bot start pipeline

@mender-test-bot
Copy link

Hello 😸 I created a pipeline for you here: Pipeline-444525489

Build Configuration Matrix

Key Value
AUDITLOGS_REV master
BUILD_BEAGLEBONEBLACK true
BUILD_CLIENT true
BUILD_QEMUX86_64_BIOS_GRUB true
BUILD_QEMUX86_64_BIOS_GRUB_GPT true
BUILD_QEMUX86_64_UEFI_GRUB true
BUILD_VEXPRESS_QEMU true
BUILD_VEXPRESS_QEMU_FLASH true
BUILD_VEXPRESS_QEMU_UBOOT_UEFI_GRUB true
CREATE_ARTIFACT_WORKER_REV master
DEPLOYMENTS_ENTERPRISE_REV master
DEPLOYMENTS_REV master
DEVICEAUTH_ENTERPRISE_REV master
DEVICEAUTH_REV master
DEVICECONFIG_REV master
DEVICECONNECT_REV master
DEVICEMONITOR_REV master
GUI_REV master
INTEGRATION_REV master
INVENTORY_ENTERPRISE_REV master
INVENTORY_REV master
IOT_MANAGER_REV master
MENDER_ARTIFACT_REV master
MENDER_CLI_REV master
MENDER_CONNECT_REV master
MENDER_REV pull/916/head
MONITOR_CLIENT_REV master
MTLS_AMBASSADOR_REV master
REPORTING_REV master
RUN_INTEGRATION_TESTS true
TENANTADM_REV master
TEST_QEMUX86_64_BIOS_GRUB true
TEST_QEMUX86_64_BIOS_GRUB_GPT true
TEST_QEMUX86_64_UEFI_GRUB true
TEST_VEXPRESS_QEMU true
TEST_VEXPRESS_QEMU_FLASH true
TEST_VEXPRESS_QEMU_UBOOT_UEFI_GRUB true
USERADM_ENTERPRISE_REV master
USERADM_REV master
WORKFLOWS_ENTERPRISE_REV master
WORKFLOWS_REV master

@kacf kacf merged commit e2e5297 into mendersoftware:master Jan 11, 2022
@kacf kacf deleted the fd_leak branch January 11, 2022 10:08
@mender-test-bot
Copy link

Hello 😸 This PR contains changelog entries. Please, verify the need of backporting it to the following release branches:
3.1.x (release 3.1.x) - 🤖 🍒
3.0.x (release 3.0.x) - 🤖 🍒
2.5.x (release 2.6.x)

@lluiscampos
Copy link
Contributor

@mender-test-bot 👇

cherry-pick to:

  • 3.1.x
  • 3.0.x

@mender-test-bot
Copy link

Hi 😺
I did my very best, and this is the result of the cherry pick operation:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants