Skip to content

[sonic-host-services] Add 'parameterized' package as a test dependency#7900

Merged
jleveque merged 1 commit intosonic-net:masterfrom
jleveque:fix_deps
Jun 17, 2021
Merged

[sonic-host-services] Add 'parameterized' package as a test dependency#7900
jleveque merged 1 commit intosonic-net:masterfrom
jleveque:fix_deps

Conversation

@jleveque
Copy link
Copy Markdown
Contributor

@jleveque jleveque commented Jun 16, 2021

Why I did it

Recently, the build started failing with messages like

2021-06-16T16:55:02.8675603Z tests/hostcfgd/hostcfgd_test.py:5: in <module>
2021-06-16T16:55:02.8676208Z     from parameterized import parameterized
2021-06-16T16:55:02.8677145Z E   ModuleNotFoundError: No module named 'parameterized'

Unit tests for hostcfgd depend on the parameterized Python package, but it was never added as a dependency to the setup.py file. This dependency was added ~3 months ago. I'm not sure why we only started seeing this failure recently.

How I did it

Add 'parameterized' package as a test dependency in setup.py for sonic-host-services package

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

@judyjoseph
Copy link
Copy Markdown
Contributor

thanks @jleveque

@prsunny
Copy link
Copy Markdown
Contributor

prsunny commented Jun 16, 2021

For 202012, there is another PR, can that be closed and cherry-pick this? @jleveque ?

@jleveque
Copy link
Copy Markdown
Contributor Author

For 202012, there is another PR, can that be closed and cherry-pick this? @jleveque ?

Thanks for the heads-up! I have closed the other PR.

@jleveque jleveque merged commit b259624 into sonic-net:master Jun 17, 2021
@jleveque jleveque deleted the fix_deps branch June 17, 2021 00:04
qiluo-msft pushed a commit that referenced this pull request Jun 17, 2021
#7900)

#### Why I did it

Recently, the build started failing with messages like

```
2021-06-16T16:55:02.8675603Z tests/hostcfgd/hostcfgd_test.py:5: in <module>
2021-06-16T16:55:02.8676208Z     from parameterized import parameterized
2021-06-16T16:55:02.8677145Z E   ModuleNotFoundError: No module named 'parameterized'
```

Unit tests for hostcfgd depend on the `parameterized` Python package, but it was never added as a dependency to the setup.py file. This dependency was added ~3 months ago. I'm not sure why we only started seeing this failure recently.

#### How I did it

Add 'parameterized' package as a test dependency in setup.py for sonic-host-services package
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
sonic-net#7900)

#### Why I did it

Recently, the build started failing with messages like

```
2021-06-16T16:55:02.8675603Z tests/hostcfgd/hostcfgd_test.py:5: in <module>
2021-06-16T16:55:02.8676208Z     from parameterized import parameterized
2021-06-16T16:55:02.8677145Z E   ModuleNotFoundError: No module named 'parameterized'
```

Unit tests for hostcfgd depend on the `parameterized` Python package, but it was never added as a dependency to the setup.py file. This dependency was added ~3 months ago. I'm not sure why we only started seeing this failure recently.

#### How I did it

Add 'parameterized' package as a test dependency in setup.py for sonic-host-services package
praveen-li pushed a commit to praveen-li/sonic-buildimage that referenced this pull request Feb 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants