Skip to content

Conversation

@ryan-jay-qiu
Copy link

@ryan-jay-qiu ryan-jay-qiu commented Jan 10, 2025

…s empty during token login in kubeedge version 1.19.1

What type of PR is this?
/kind bug

What this PR does / why we need it:

When logging into the dashboard with kubeedge version 1.91.1, an error occurs and the system cannot be accessed. It is possible that the version is not compatible, so please fix it

629D3B28-C6F0-4a33-9224-993BB1DBDB88
C5FD6EA5-0BE2-4511-9AB7-E1DF599417AA
FC2A349F-9903-4583-9F52-6169C6DDD2F7

Which issue(s) this PR fixes:

Fixes #
fix:Resolve the issue of the getServiceAccountName function parsing as empty during token login in kubeedge version 1.19.1
Special notes for your reviewer:

Does this PR introduce a user-facing change?:


@kubeedge-bot kubeedge-bot added the kind/bug Categorizes issue or PR as related to a bug. label Jan 10, 2025
@kubeedge-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign kevin-wangzefeng after the PR has been reviewed.
You can assign the PR to them by writing /assign @kevin-wangzefeng in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubeedge-bot
Copy link
Collaborator

Welcome @ryan-jay-qiu! It looks like this is your first PR to kubeedge/dashboard 🎉

@kubeedge-bot kubeedge-bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 10, 2025
@ryan-jay-qiu
Copy link
Author

This was referenced Jan 10, 2025
…s empty during token login in kubeedge version 1.19.1

Signed-off-by: ryan-jay-qiu <[email protected]>
Copy link
Contributor

@wbc6080 wbc6080 left a comment

Choose a reason for hiding this comment

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

PTAL @ghosind

@ghosind
Copy link
Member

ghosind commented Apr 7, 2025

/lgtm

@kubeedge-bot kubeedge-bot added the lgtm Indicates that a PR is ready to be merged. label Apr 7, 2025
@kubeedge-bot
Copy link
Collaborator

@ryan-jay-qiu: PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@kubeedge-bot kubeedge-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants