Skip to content

Fix slice init length in fetcher pod spec addition #3018

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 30, 2024
Merged

Fix slice init length in fetcher pod spec addition #3018

merged 1 commit into from
Sep 30, 2024

Conversation

cuishuang
Copy link
Contributor

Description

Fix slice init length

Which issue(s) this PR fixes:

Fixes #

Testing

Checklist:

  • I ran tests as well as code linting locally to verify my changes.
  • I have done manual verification of my changes, changes working as expected.
  • I have added new tests to cover my changes.
  • My changes follow contributing guidelines of Fission.
  • I have signed all of my commits.

@soharab-ic soharab-ic added the bug label Sep 29, 2024
@soharab-ic
Copy link
Contributor

soharab-ic commented Sep 29, 2024 •
edited
Loading

@cuishuang Thank you for the PR.
Please sign your commit.

@cuishuang
Fix slice init length
a52f76b 
Signed-off-by: cuishuang <imcusg@gmail.com>
@cuishuang
Copy link
Contributor Author

@cuishuang Thank you for the PR. Please sign your commit.

Thanks. Signed.

@sanketsudake sanketsudake changed the title Fix slice init length Fix slice init length in fetcher pod spec addition Sep 30, 2024
@sanketsudake sanketsudake merged commit 2db9db0 into fission:main Sep 30, 2024
9 checks passed
@cuishuang cuishuang mentioned this pull request Oct 14, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sanketsudake sanketsudake sanketsudake approved these changes

@soharab-ic soharab-ic soharab-ic approved these changes

Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cuishuang @soharab-ic @sanketsudake