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.

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

Cast uptime values to int before processing #1343

Merged
merged 1 commit into from
Dec 30, 2020
Merged

Cast uptime values to int before processing #1343

merged 1 commit into from
Dec 30, 2020

Conversation

paketb0te
Copy link
Contributor

I ran into a TypeError here: TypeError: unsupported operand type(s) for +=: 'int' and 'str'
Casting the uptime values (days, hours, mins, seconds) to int before processing fixes this issue.

@paketb0te
Cast uptime values to int before processing
a590137 
I ran into a TypeError here: TypeError: unsupported operand type(s) for +=: 'int' and 'str'
Casting the uptime values (days, hours, mins, seconds) to int before processing fixes this issue.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 0.0% when pulling a590137 on paketb0te:patch-1 into 5a2adcb on napalm-automation:develop.

@mirceaulinic mirceaulinic added this to the 3.3.0 milestone Dec 30, 2020
@mirceaulinic mirceaulinic merged commit 6802a04 into napalm-automation:develop Dec 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ktbyers ktbyers ktbyers approved these changes

Assignees
No one assigned
Labels
bug get_facts nxos
Projects
None yet
Milestone
3.3.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@paketb0te @coveralls @ktbyers @mirceaulinic