-
Notifications
You must be signed in to change notification settings - Fork 60
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
Added a new Definition to upload firmware images greater than 2GB in size #243
Conversation
Adding new definitions to read a file in chunks , creating URI and to upload firmware
Adding a new definition to upload firmware for file size greater than 2 GB
Co-authored-by: Vikrant Balyan <[email protected]>
ucsmsdk/utils/ucsguilaunch.py
Outdated
@@ -56,7 +56,7 @@ def ucs_gui_launch(handle, need_url=False): | |||
|
|||
jnlp_file = None | |||
try: | |||
ucsm_gui_url = "%s/ucsm/ucsm.jnlp" % handle.uri |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How is this change related to this PR or firmware upload in general?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an another issue -
#241
https://jira-eng-sjc11.cisco.com/jira/browse/UCSES-27 (Please refer attachment)
Does it required to create a new Fork and raise PR ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can create new branches in your fork for each bug fix and submit individual PRs from each of your bug fix branches. Do not mix multiple unrelated changes in a single PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure,
Added a new definition to read the file in chunks, to create chunks URI and to upload firmware