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

1ESPT conversion cleanup #14531

Open
chcosta opened this issue Feb 29, 2024 · 5 comments
Open

1ESPT conversion cleanup #14531

chcosta opened this issue Feb 29, 2024 · 5 comments
Labels
1ES Template Migration Issues related to 1ES template migration

Comments

@chcosta
Copy link
Member

chcosta commented Feb 29, 2024

@chcosta chcosta changed the title Investigate whether Arcade CodeQL template is still required 1ESPT conversion cleanup Feb 29, 2024
@riarenas
Copy link
Member

riarenas commented Feb 29, 2024

Adding a few more items

  • Clean up any conditional that mentions running a PR, running as public, or running in the public project

  • Understand whether we still need to generate SBOMs with our own templates or can solely rely on the template injected sbom generation. This might not be possible, as the Microbuild guidance is to keep using the manifest generator task...

    SBOM: The Migration Tooling will remove the SBOM Manifest Builder Task. Please put this back. The SBOM tasks in the 1ES PT does not conform to the way that we package up SBOM's into manifest files in MicroBuild.

@dougbu
Copy link
Member

dougbu commented Feb 29, 2024

re the CodeQL point: parameters.PackageVersion is a lousy name anyhow. we should at least fix that if the hook is still needed

@akoeplinger
Copy link
Member

akoeplinger commented Mar 1, 2024

Understand whether we still need to generate SBOMs with our own templates or can solely rely on the template injected sbom generation. This might not be possible, as the Microbuild guidance is to keep using the manifest generator task...

if we keep our own SBOM task then we need to bump the package version since it wasn't bumped in main: #14511

@riarenas
Copy link
Member

riarenas commented Mar 1, 2024

As a more general cleanup point:

  • the public templates should only do public things
  • the official templates should only do official things

@riarenas
Copy link
Member

riarenas commented Mar 1, 2024

Understand whether we still need to generate SBOMs with our own templates or can solely rely on the template injected sbom generation. This might not be possible, as the Microbuild guidance is to keep using the manifest generator task...

if we keep our own SBOM task then we need to bump the package version since it wasn't bumped in main: #14511

Especially because a certain @riarenas said we shouldn't update it in main yet😛

@riarenas riarenas added the 1ES Template Migration Issues related to 1ES template migration label Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1ES Template Migration Issues related to 1ES template migration
Projects
None yet
Development

No branches or pull requests

4 participants