Ironic

Warning in api-ref tox environment

Bug #1581778 reported by Daniel Gonzalez Nothnagel on 2016-05-14

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Cinder	Fix Released	Medium	Daniel Gonzalez Nothnagel	Cinder newton-1 "n1"
	Ironic	Fix Released	Medium	Jim Rollenhagen	Ironic ongoing

Bug Description

When running the api-ref tox environment the following warning is issued:

WARNING:test command found but not installed in testenv
cmd: /bin/rm
env: /Users/user/repos/ironic/.tox/api-ref
Maybe you forgot to specify a dependency? See also the whitelist_externals envconfig setting.

Tags:

Daniel Gonzalez Nothnagel (dgonzalez) on 2016-05-14

Changed in ironic:
assignee:	nobody → Daniel Gonzalez Nothnagel (dgonzalez)
Changed in cinder:
assignee:	nobody → Daniel Gonzalez Nothnagel (dgonzalez)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-05-14: Fix proposed to ironic (master)

Fix proposed to branch: master
Review: https://review.openstack.org/316362

Changed in ironic:
status:	New → In Progress
Changed in cinder:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-05-14: Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/316363

Sheel Rana (ranasheel2000) on 2016-05-14

Changed in cinder:
importance:	Undecided → Medium

venkatamahesh (venkatamaheshkotha) on 2016-05-16

Changed in ironic:
importance:	Undecided → Medium
Changed in cinder:
milestone:	none → newton-1
Changed in ironic:
milestone:	none → ongoing

Mitsuhiro Tanino (mitsuhiro-tanino) on 2016-05-16

tags:

added: api-ref

OpenStack Infra (hudson-openstack) on 2016-05-16

Changed in ironic:
assignee:	Daniel Gonzalez Nothnagel (dgonzalez) → Jim Rollenhagen (jim-rollenhagen)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-05-17: Fix merged to ironic (master)

Reviewed: https://review.openstack.org/316793
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=6fbf5c55167672dd95640743b5ae36ee61a7880f
Submitter: Jenkins
Branch: master

commit 6fbf5c55167672dd95640743b5ae36ee61a7880f
Author: Jim Rollenhagen <email address hidden>
Date: Mon May 16 12:18:26 2016 +0000

Eliminate warnings about rm in api-ref build

    This currently gives a warning about the 'rm' command not being
    installed in the venv. Whitelist bash and run rm under bash (like we do
    with other external commands) to squash this warning.

Closes-Bug: #1581778
Change-Id: Id26926ad7a75b94f0f57ea3c2083312c66663b34

Changed in ironic:
status:	In Progress → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-05-17: Fix merged to cinder (master)

Reviewed: https://review.openstack.org/316363
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=c2f2eeb394d03df6d7f9147bb6fdcbaceb777117
Submitter: Jenkins
Branch: master

commit c2f2eeb394d03df6d7f9147bb6fdcbaceb777117
Author: Daniel Gonzalez <email address hidden>
Date: Sat May 14 11:30:24 2016 +0200

Whitelist 'rm' in api-ref tox environment

    Running the api-ref environment currently warns that 'rm' is not
    installed in testenv. This patch removes the warning by whitelisting
    the rm command.

Change-Id: I4072491713a38b11d9f55b26a8ba0a3cf49445f0
Closes-Bug: #1581778

Changed in cinder:
status:	In Progress → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-05-19: Change abandoned on ironic (master)

Change abandoned by Daniel Gonzalez Nothnagel (<email address hidden>) on branch: master
Review: https://review.openstack.org/316362
Reason: https://review.openstack.org/#/c/316793/ has merged.
I will therefore abandon this patch.