Skip to content

DAOS-623 build: Remove debian files #17050

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.

Sign up for GitHub

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

Jump to bottom
Merged
jolivier23 merged 1 commit into from
Oct 31, 2025
Merged

DAOS-623 build: Remove debian files #17050

jolivier23 merged 1 commit into from
Oct 31, 2025

Conversation

@jolivier23
Copy link
Contributor

@jolivier23 jolivier23 commented Oct 29, 2025

The move to fpm renders them obsolete.

Skip-test: true

Change-Id: I7917b5d83ebdae10cb1da6af45b33cdebe999104

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

@jolivier23
DAOS-623 build: Remove debian files
74e4127 
The move to fpm renders them obsolete.

Skip-test: true

Change-Id: I7917b5d83ebdae10cb1da6af45b33cdebe999104
Signed-off-by: Jeff Olivier <jeffolivier@google.com>
@github-actions
Copy link

github-actions bot commented Oct 29, 2025

Ticket title is 'Generic ticket for minor code cleanup and improvement'
Status is 'Resolved'
Labels: 'request_for_2.6.4,request_for_2.6.5'
https://daosio.atlassian.net/browse/DAOS-623

@jolivier23 jolivier23 mentioned this pull request Oct 29, 2025
Open
6 tasks
knard38
knard38 approved these changes Oct 30, 2025
View reviewed changes
Copy link
Contributor

@knard38 knard38 left a comment

Choose a reason for hiding this comment

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

LGTM

@jolivier23 jolivier23 requested a review from a team October 30, 2025 21:33
@jolivier23
Copy link
Contributor Author

jolivier23 commented Oct 30, 2025

This is just dead code removal so that is why I added gatekeeper early

@jolivier23 jolivier23 merged commit c3e8ea7 into master Oct 31, 2025
38 checks passed
@jolivier23 jolivier23 deleted the jvolivie/debian branch October 31, 2025 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ryon-jensen ryon-jensen ryon-jensen approved these changes

@knard38 knard38 knard38 approved these changes

@johannlombardi johannlombardi Awaiting requested review from johannlombardi

@daltonbohning daltonbohning Awaiting requested review from daltonbohning

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jolivier23 @ryon-jensen @knard38