vmware / vmware/pyvmomi

Python 3 string formatting bug in Iso8601.py

Open
#169 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug help wanted low hanging fruit
Dominant language
Python
Stars
2.3k
Forks
763
PR merge metrics
No merged PRs in 30d

Description

There is at least one place in Iso8601.py that does not include python 3 compatible string formatting.

This indicates a poor testing coverage over date formatting issues.

  • Devise a test that exercises this section of code by only using exposed VMOMI API ... that is by sending or receiving a vSphere command.
  • After devising the test then fix the code.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with pyVmomi/Iso8601.py around line 198 and trace how the affected date-formatting path is reached through the exposed VMOMI API. Devise a test that sends or receives a vSphere command using only that API, then use it to verify the Python 3 formatting fix. Done means the test exercises this path and the formatting works under Python 3.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.