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

(#2190153) pstore: fix crash and forward dummy arguments instead of NULL #385

Merged
merged 3 commits into from
May 10, 2023

Conversation

msekletar
Copy link
Member

@msekletar msekletar commented Apr 27, 2023

Resolves: #2190153

@mergify mergify bot added the pr/needs-ci Formerly needs-ci label Apr 27, 2023
@github-actions
Copy link

github-actions bot commented Apr 27, 2023

Tracker - 2190153

The following commits meet all requirements

commit upstream
7579e38 - pstore: fix crash and forward dummy arguments instead of NULL rhel-only
c205125 - ci: workflow for gathering metadata for source-git automation rhel-only
3a8bf07 - ci: first part of the source-git automation - commit linter rhel-only

@dtardon
Copy link
Member

dtardon commented Apr 27, 2023

@jamacku Apparently bot has eaten the bug reference in the description.

msekletar and others added 3 commits April 27, 2023 12:35
[msekleta: in our version of systemd "const char path*" argument of
path_join() can't be NULL. Here we don't really want any subdirs paths
passed into move_file(), but we can't just pass NULL pointers because
they will be forwarded to path_join(). Hence, let's just pass "/"
instead.]

rhel-only

Related: #2190153
Workflow gathers metadata like pull request numbers and information about commits.
This metadata is used for commit validation and other actions.
This workflow also triggers for rest of the source-git automation workflows.

rhel-only

Related: #2190153
Add a GitHub Workflow that is triggered on `workflow_run` events.
It uses metadata provided by `redhat-plumbers-in-action/gather-pull-request-metadata`
GitHub Action to get the PR number and the commit metadata.
The commit metadata is then used to check if the commit message contains
all required information (tracker and upstream reference). GitHub Action
responsible for commit verification `redhat-plumbers-in-action/advanced-commit-linter`
is configured via the `advanced-commit-linter.yml` file.

rhel-only

Related: #2190153
@jamacku
Copy link
Member

jamacku commented Apr 27, 2023

@jamacku Apparently bot has eaten the bug reference in the description.

Sorry, ^ and $ are a part of regex and should not be added to the comment. I'll update GA.

@redhat-plumbers redhat-plumbers deleted a comment from github-actions bot Apr 27, 2023
Copy link
Member

@dtardon dtardon left a comment

Choose a reason for hiding this comment

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

LGTM

@systemd-rhel-bot systemd-rhel-bot removed the pr/needs-review Formerly needs-review label Apr 27, 2023
jamacku added a commit to jamacku/advanced-commit-linter that referenced this pull request Apr 27, 2023
jamacku added a commit to redhat-plumbers-in-action/advanced-commit-linter that referenced this pull request Apr 27, 2023
@systemd-rhel-bot systemd-rhel-bot changed the title (#2190153) pstore: fix crash and forward dummy arguments instead of NULL (#2190153) (#2190153) pstore: fix crash and forward dummy arguments instead of NULL May 1, 2023
@jamacku jamacku changed the title (#2190153) (#2190153) pstore: fix crash and forward dummy arguments instead of NULL (#2190153) pstore: fix crash and forward dummy arguments instead of NULL May 3, 2023
@systemd-rhel-bot systemd-rhel-bot added tracker/unapproved Formerly needs-acks and removed tracker/unapproved Formerly needs-acks labels May 6, 2023
@jamacku jamacku merged commit e08bdd2 into redhat-plumbers:rhel-8.8.0 May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants