Skip to content

recipes-tests/converged-security-suite: add - #7

Open
macpijan wants to merge 3 commits into
mainfrom
converged-secruity-suite
Open

recipes-tests/converged-security-suite: add#7
macpijan wants to merge 3 commits into
mainfrom
converged-secruity-suite

Conversation

@macpijan

@macpijan macpijan commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

This should replace: https://github.com/Dasharo/meta-dts/tree/main/meta-dts-distro/recipes-tests/converged-security-suite

Although not strictly dasharo project, it will make us easier to reuse this recipe. Right now, I could not reuse it under meta-trenchboot, if it was present in meta-dts.

Binaries are properly packaged:

macpijan in ~/projects/trenchboot/yocto/build/tmp/work/core2-64-tb-linux/converged-security-suite/2.8.1 λ tree packages-split                                
packages-split
├── converged-security-suite
├── converged-security-suite-amd
│   └── usr
│       └── bin
│           └── amd-suite
├── converged-security-suite-bg
│   └── usr
│       └── bin
│           ├── bg-prov
│           └── bg-suite
├── converged-security-suite-dbg
│   └── usr
│       └── bin
├── converged-security-suite-dev
├── converged-security-suite-doc
├── converged-security-suite-locale
├── converged-security-suite-src
├── converged-security-suite-staticdev
├── converged-security-suite-tools
│   └── usr
│       └── bin
│           └── pcr0tool
└── converged-security-suite-txt
    └── usr
        └── bin
            ├── txt-prov
            └── txt-suite

Not runtime tested yet.

@macpijan
macpijan requested a review from DaniilKl July 9, 2025 10:40
macpijan added 2 commits July 9, 2025 13:01
Signed-off-by: Maciej Pijanowski <maciej.pijanowski@3mdeb.com>
Signed-off-by: Maciej Pijanowski <maciej.pijanowski@3mdeb.com>
@macpijan
macpijan force-pushed the converged-secruity-suite branch from 464eec0 to eccba1e Compare July 9, 2025 11:01
@macpijan

macpijan commented Jul 9, 2025

Copy link
Copy Markdown
Contributor Author

Unfortunately, it seems that those tools are broken, and it's pointless to include recipe for this version:

The last known working version (at least for txt-suite) is from 4 years ago and we might include it here: https://github.com/9elements/converged-security-suite/releases/tag/v2.6.0

Signed-off-by: Maciej Pijanowski <maciej.pijanowski@3mdeb.com>
@macpijan
macpijan force-pushed the converged-secruity-suite branch from 269d13a to ab8695d Compare September 28, 2025 11:14

@DaniilKl DaniilKl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks ok. I have not tested it in any system just verified if the package builds correctly.

install -d ${D}${bindir}
# install -m 0755 ${S}/amd-suite ${D}${bindir}
# install -m 0755 ${S}/bg-prov ${D}${bindir}
# install -m 0755 ${S}/bg-suite ${D}${bindir}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What do you think about adding bg-suite binary to the layer as was done here https://github.com/Dasharo/meta-dts/blob/2a634e361986aa16286469efd8c879191027458e/meta-dts-distro/recipes-tests/converged-security-suite/bg-suite_2.7.0.bb#L7 ? Consider adding it so we will include this package in DTS later.

@macpijan macpijan Oct 24, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

What do you think about adding bg-suite binary to the layer

I think that adding any binary to the layer is a truly terrible idea.
We can fetch this binary via SRC_URI from somewhere - if we cannot build it in yocto.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Or the place where this binary was taken from.

@macpijan macpijan Oct 24, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

If there is no upstream release, you can possibly upload this binary to GH release in our fork.

@macpijan

Copy link
Copy Markdown
Contributor Author

Looks ok. I have not tested it in any system just verified if the package builds correctly.

I tested in https://github.com/zarhus/meta-trenchboot/blob/master/recipes-tests/trenchboot-hcl-report/trenchboot-hcl-report/trenchboot-hcl-report.sh#L221 many times already

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants