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

Updating CRI roles #70

Merged
merged 4 commits into from
Oct 5, 2020
Merged

Updating CRI roles #70

merged 4 commits into from
Oct 5, 2020

Conversation

xunholy
Copy link
Member

@xunholy xunholy commented Sep 28, 2020

Description

  • Removing hardcoded LSB
  • General Fixes

Type Of Change

To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

  • Docs
  • Installation
  • Performance and Scalability
  • Security
  • Test and/or Release
  • User Experience

Issue Ref (Optional)

Which issue(s) this PR fixes (optional, using fixes #(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged): Fixes #

Notes

Add special notes for your reviewer here.

Signed-off-by: Michael Fornaro <20387402+xUnholy@users.noreply.github.com>
Signed-off-by: Michael Fornaro <20387402+xUnholy@users.noreply.github.com>
@xunholy xunholy mentioned this pull request Sep 28, 2020
9 tasks
Signed-off-by: Michael Fornaro <20387402+xUnholy@users.noreply.github.com>
@crutonjohn
Copy link
Member

this LGTM.

i'll leave it to you to merge.

- cri
# - networking
Copy link
Member

Choose a reason for hiding this comment

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

i appreciate the name change here

@xunholy xunholy merged commit 477c934 into master Oct 5, 2020
@xunholy xunholy deleted the feature/cri branch October 5, 2020 12:20
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.

None yet

2 participants