Skip to content
This repository has been archived by the owner on May 30, 2023. It is now read-only.

Drop mantle and kola-data from the SDK #1827

Merged
merged 1 commit into from
Apr 21, 2022
Merged

Drop mantle and kola-data from the SDK #1827

merged 1 commit into from
Apr 21, 2022

Conversation

pothos
Copy link
Contributor

@pothos pothos commented Apr 20, 2022

With the new mantle container image referenced by the scripts repo we
don't need the mantle copy in the SDK anymore.
Drop the mantle package and the unused kola-data package.

How to use

pick for all channels

Goes with flatcar/scripts#280

Testing done

passed

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)

@jepio
Copy link
Contributor

jepio commented Apr 20, 2022

Aren't bits of mantle used by various things in scripts?

@pothos
Copy link
Contributor Author

pothos commented Apr 20, 2022

If yes, they should use the container image, except for the original Jenkins pipeline that uses the binaries from Jenkins

@pothos
Copy link
Contributor Author

pothos commented Apr 20, 2022

looks like jenkins/sdk.sh and jenkins/toolchains.sh need to be touched to use the Jenkins binaries

@jepio
Copy link
Contributor

jepio commented Apr 20, 2022

We're dismantling the SDK?

@pothos
Copy link
Contributor Author

pothos commented Apr 20, 2022

I'll run a test build with flatcar/scripts#280

Copy link
Contributor

@tormath1 tormath1 left a comment

Choose a reason for hiding this comment

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

That looks good to me - we should maybe go through the documentation to search for Mantle SDK mentions and update them consequently.

changelog/changes/2022-04-20-mantle-sdk.md Outdated Show resolved Hide resolved
With the new mantle container image referenced by the scripts repo we
don't need the mantle copy in the SDK anymore.
Drop the mantle package and the unused kola-data package.
@pothos pothos merged commit 07dbfed into main Apr 21, 2022
@pothos pothos deleted the kai/drop-kola-from-sdk branch April 21, 2022 08:20
pothos added a commit that referenced this pull request Apr 21, 2022
Drop mantle and kola-data from the SDK
pothos added a commit that referenced this pull request Apr 21, 2022
Drop mantle and kola-data from the SDK
pothos added a commit that referenced this pull request Apr 21, 2022
Drop mantle and kola-data from the SDK
pothos added a commit that referenced this pull request Apr 21, 2022
Drop mantle and kola-data from the SDK
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants