test suite reviews and discussions
 help / color / Atom feed
* [dts] Question about the skip setup flag and a possible addition to bug 515
@ 2020-07-29 11:30 Owen Hilyard
  0 siblings, 0 replies; only message in thread
From: Owen Hilyard @ 2020-07-29 11:30 UTC (permalink / raw)
  To: dts; +Cc: Lincoln Lavoie, David Liu, Daniel Kirichok, david.marchand

[-- Attachment #1: Type: text/plain, Size: 1683 bytes --]

Hello all,

During a recent internal meeting, we (the IOL team) were discussing
bringing new test cases into the CI environment. We found that many of the
test cases which recompile DPDK seem to break testpmd for all of the tests
after them. As far as we can tell, this is because some of these tests
leave testpmd in a location different than the default once they are
compiled, resulting in the executable not being able to be found. We wanted
to reach out and ask if the skip setup flag ('-s', '--skip-setup') was
intended to stop recompilation of dpdk during the time that test cases are
running. If it is, then it seems many of the tests which recompile dpdk do
not follow this convention, and if it is not then those test cases might
need to be fixed. It is also possible that there is a misconfiguration or
misunderstanding of the root cause on our side. If it is not one of those
cases and this is both behavior that is expected given proper configuration
but not behavior which is desired, then I think I have a relatively
painless solution. This solution would be to add a check for the flag in
the build_install_dpdk function and, if it is present, assumes that
everything is compiled correctly and exits the function early. This would
only involve adding a few lines for the check and then whatever is required
to access the flags deeper into dts.

Where this becomes a possible addition to bug 515
<https://bugs.dpdk.org/show_bug.cgi?id=515>, is that doing this could break
many of the tests which edit dpdk source directly, since the compilation
they expect would not occur. As such, this would either need to be its own
bug to be fixed after 515, or tacked on to 515.

[-- Attachment #2: Type: text/html, Size: 1768 bytes --]

<div dir="ltr">Hello all,<div><br></div><div>During a recent internal meeting, we (the IOL team) were discussing bringing new test cases into the CI environment. We found that many of the test cases which recompile DPDK seem to break testpmd for all of the tests after them. As far as we can tell, this is because some of these tests leave testpmd in a location different than the default once they are compiled, resulting in the executable not being able to be found. We wanted to reach out and ask if the skip setup flag (&#39;-s&#39;, &#39;--skip-setup&#39;) was intended to stop recompilation of dpdk during the time that test cases are running. If it is, then it seems many of the tests which recompile dpdk do not follow this convention, and if it is not then those test cases might need to be fixed. It is also possible that there is a misconfiguration or misunderstanding of the root cause on our side. If it is not one of those cases and this is both behavior that is expected given proper configuration but not behavior which is desired, then I think I have a relatively painless solution. This solution would be to add a check for the flag in the build_install_dpdk function and, if it is present, assumes that everything is compiled correctly and exits the function early. This would only involve adding a few lines for the check and then whatever is required to access the flags deeper into dts. <br><br>Where this becomes a possible addition to <a href="https://bugs.dpdk.org/show_bug.cgi?id=515">bug 515</a>, is that doing this could break many of the tests which edit dpdk source directly, since the compilation they expect would not occur. As such, this would either need to be its own bug to be fixed after 515, or tacked on to 515. </div></div>

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, back to index

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-29 11:30 [dts] Question about the skip setup flag and a possible addition to bug 515 Owen Hilyard

test suite reviews and discussions

Archives are clonable:
	git clone --mirror http://inbox.dpdk.org/dts/0 dts/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dts dts/ http://inbox.dpdk.org/dts \
		dts@dpdk.org
	public-inbox-index dts


Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dts


AGPL code for this site: git clone https://public-inbox.org/ public-inbox