Skip to content

test_lava.py: Modify example in function comment of get_device_type_by_name.#545

Closed
wangmingyu84 wants to merge 1 commit into
kernelci:masterfrom
wangmingyu84:master
Closed

test_lava.py: Modify example in function comment of get_device_type_by_name.#545
wangmingyu84 wants to merge 1 commit into
kernelci:masterfrom
wangmingyu84:master

Conversation

@wangmingyu84
Copy link
Copy Markdown
Contributor

The example of the function does not match the code.

Signed-off-by: Wang Mingyu wangmy@cn.fujitsu.com

…y_name.

The example of the function does not match the code.

Signed-off-by: Wang Mingyu <wangmy@cn.fujitsu.com>
@gctucker gctucker requested a review from a team November 19, 2020 09:30
@gctucker gctucker added staging-skip Don't test automatically on staging.kernelci.org and removed staging-skip Don't test automatically on staging.kernelci.org labels Jan 5, 2021
gctucker added a commit to kernelci/kernelci-deploy that referenced this pull request Jan 5, 2021
Apparently it's possible to create a PR from an "unknown" repository,
such as this one:

  kernelci/kernelci-core#545

Since the user name is currently derived from the source repository,
it can't be verified whether this is from a trusted user.  Skip the
PRs with an unknown repository for now.

Signed-off-by: Guillaume Tucker <guillaume.tucker@collabora.com>
@gctucker gctucker closed this Jan 12, 2021
@gctucker gctucker deleted the branch kernelci:master January 12, 2021 21:49
@gctucker
Copy link
Copy Markdown
Collaborator

Sorry this got closed by accident when the master branch was renamed to main. I think it was because the incoming repository for this PR is unknown.

@gctucker
Copy link
Copy Markdown
Collaborator

@wangmingyu84 I've now simplified the code: #590. That function was from a legacy implementation before the kernelci.lab.lava module was created. There's now a method in the LAVA class using a plain Python dictionary to translate aliases into device types. Thanks for reporting the issue, it made it clear that there was a problem in the code.

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.

2 participants