Skip to content

Conversation

@DixitAakash
Copy link
Contributor

No description provided.

@DixitAakash
Copy link
Contributor Author

build amko

@DixitAakash
Copy link
Contributor Author

FT smoke results.
FT_smok_amko.txt

@arihantg
Copy link
Contributor

arihantg commented Jan 7, 2026

Do we need changes for session refresh here also as done in ako?

@DixitAakash
Copy link
Contributor Author

DixitAakash commented Jan 7, 2026

Do we need changes for session refresh here also as done in ako?

@arihantg I did not see this error "Rest request error, returning to caller" being handled explicitly in amko, hence did not make any change. The change was required in ako as it is handled explicitly, and sdk now stopped bypassing the response. Instead, the response from controller is directly returned which is different from above error. Do we need handling for this error in amko as well?

Copy link
Contributor

@arihantg arihantg left a comment

Choose a reason for hiding this comment

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

LGTM.

@DixitAakash DixitAakash merged commit b97f6b3 into vmware:master Jan 9, 2026
3 checks passed
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.

3 participants