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

cmd/cork,sdk: remove --repo-branch option from cork #283

Merged
merged 2 commits into from
Jan 28, 2022

Conversation

dongsupark
Copy link
Member

Now that repo 2.8 is available in all Flatcar channels, the option --repo-branch is not needed any more.
Delete the option, and simply use the master branch of the git repo.

This reverts commit 0713901.

Testing done

local build passed

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

Now that repo 2.8 is available in all Flatcar channels, the option
`--repo-branch` is not needed any more.
Delete the option, and simply use the `master` branch of the git repo.

This reverts commit 0713901.
@dongsupark dongsupark added the enhancement New feature or request label Jan 28, 2022
@dongsupark dongsupark requested a review from a team January 28, 2022 11:55
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.

LGTM !
IIRC repo-2.8 has also landed in the LTS SDK ?

CHANGELOG.md Outdated Show resolved Hide resolved
@dongsupark dongsupark force-pushed the dongsu/cork-repo-branch-revert branch from a5cc020 to add02cc Compare January 28, 2022 13:33
@dongsupark
Copy link
Member Author

IIRC repo-2.8 has also landed in the LTS SDK ?

Yes, repo 2.8 is already in SDK of LTS 2605.24.1.

Copy link
Member

@krnowak krnowak left a comment

Choose a reason for hiding this comment

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

Thanks for doing it. Does some jenkins os code need to be updated too?

@dongsupark
Copy link
Member Author

Does some jenkins os code need to be updated too?

No, I don't think we need to change anything in jenkins-os.

@dongsupark dongsupark force-pushed the dongsu/cork-repo-branch-revert branch from add02cc to ec1300b Compare January 28, 2022 14:32
@dongsupark dongsupark merged commit bdb2a8a into flatcar-master Jan 28, 2022
@dongsupark dongsupark deleted the dongsu/cork-repo-branch-revert branch January 28, 2022 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants