<div dir="ltr">Hi,<div><br></div><div>We have tested test-requests from fserver after pulling the latest changes.</div><div>Tests are working fine and also updates test results on fserver.</div><div><br></div><div>Thanks and regards,</div><div>Pooja More</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 5, 2021 at 4:02 AM <<a href="mailto:Tim.Bird@sony.com">Tim.Bird@sony.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hey everyone,<br>
<br>
I'm back from vacation.  Sorry this took so long to respond to,<br>
but I was away from work since Dec. 22.  Please see my comments<br>
inline below.<br>
<br>
> -----Original Message-----<br>
> From: Pooja Sanjay More <<a href="mailto:pooja.sm@pathpartnertech.com" target="_blank">pooja.sm@pathpartnertech.com</a>><br>
> <br>
> Hi Tim,<br>
> <br>
> We checked with the IT team on the result upload issue and it was indeed a firewall<br>
> configuration which was blocking it. This issue has been resolved now and we have<br>
> successfully verified the result object updates on the server.<br>
<br>
That is very good to hear!  Thanks for getting this resolved on your end.<br>
<br>
> <br>
> <br>
> Logs for a test :<br>
> <br>
> <br>
> root@cpu-366U:~# ftc put-request pptlab:rpi Functional.hello_world<br>
> Request was accepted by the server.<br>
> request_id=request-2020-12-24_12:39:23.93-pptlab:rpi<br>
> <br>
> root@cpu-366U:~# ftc run-request --put-run request-2020-12-24_12:39:23.93-pptlab:rpi<br>
> Trying to get request 'request-2020-12-24_12:39:23.93-pptlab:rpi' from server<br>
> Executing test Functional.hello_world on board rpi (using spec default)<br>
> Running test 'Functional.hello_world' on board 'rpi' using spec 'default'<br>
> ===== doing fuego phase: pre_test =====<br>
> ===== doing fuego phase: build =====<br>
> The test is already built<br>
> Fuego test_build duration=0 seconds<br>
> ===== doing fuego phase: deploy =====<br>
> ===== doing fuego phase: snapshot =====<br>
> Firmware revision: 5.4.83-v7-52+<br>
> ===== doing fuego phase: run =====<br>
> -------------------------------------------------<br>
> Hello World!<br>
> <br>
> This is a test program, to demonstrate test success and failure<br>
> Test result: SUCCESS<br>
> -------------------------------------------------<br>
> ===== doing fuego phase: post_test =====<br>
> ===== doing fuego phase: processing =====<br>
> log_compare: pattern 'SUCCESS' found 1 times (expected greater or equal than 1)<br>
> chart config not found. Using default values.<br>
> Fuego: requested test phases complete!<br>
> Packaging run 'Functional.hello_world-default-130-rpi'<br>
> run/<br>
> run/machine-snapshot.txt<br>
> run/syslog.before.txt<br>
> run/devlog.txt<br>
> run/testlog.txt<br>
> run/run.json<br>
> run/syslog.after.txt<br>
> run/prolog.sh<br>
> run/spec.json<br>
> run/build.xml<br>
> run/consolelog.txt<br>
> Run packaged successfully, and is at: /tmp/run-Functional.hello_world-default-130-on-pptlab:rpi.frp<br>
> Run package run-Functional.hello_world-default-130-on-pptlab:rpi.frp was accepted by the server.<br>
> Run Functional.hello_world-default-130-rpi put to server<br>
<br>
This looks good.<br>
<br>
> <br>
> Fserver Output:<br>
> <br>
> <br>
> <br>
> <br>
> We ran multiple tests and found that when there are tests which do not have test.yaml files<br>
> give an error for  "eprint".<br>
> <br>
> <br>
> <br>
> <br>
> <br>
> eprint() was not defined in ftc. We added it to ftc then it is working fine.<br>
> <br>
> Changes in ftc:<br>
> <br>
> diff --git a/scripts/ftc b/scripts/ftc<br>
> index ae9ab2f..e70847d 100755<br>
> --- a/scripts/ftc<br>
> +++ b/scripts/ftc<br>
> @@ -616,6 +616,9 @@ def wprint(msg):<br>
>      global ll_warning<br>
>      if ll_warning:<br>
>          print("### WARNING: " + msg)<br>
> +# always print errors<br>
> +def eprint(msg):<br>
> +    print("!!! ERROR: " + msg)<br>
> <br>
>  class config_class:<br>
>      def __init__(self, config_path):<br>
> <br>
> Patch file is attached below.<br>
<br>
I applied the patch.  It was not a well-formed patch for integration with<br>
the project.  It was missing the author, commit text, signed-off-by line<br>
and other meta-information required for a good commit.<br>
<br>
I added this material, and added my own "Signed-off-by" line.  Please<br>
note that I added a "Signed-off-by" line for you (Pooja).  Please let<br>
me know if there is any problem with this.  Since the code looks like it<br>
was taken from parser/common.py, I guessed that it was OK to add<br>
your signed-off-by for the patch.<br>
<br>
I found some other bugs in my testing, and added some other patches.<br>
These have all been pushed to the master branch on bitbucket.<br>
<br>
Please 'git pull' the latest material, and let me know if you see any problems.<br>
<br>
It is really nice to get these bugs fixed.  Thanks<br>
for the testing and the patch!!<br>
<br>
I have some other comments that I'll put in another thread.<br>
<br>
Thanks,<br>
 -- Tim<br>
<br>
</blockquote></div>

<br>




<p><span lang="EN-IN">This
message contains confidential information and is intended only for the
individual(s) named.</span><span lang="EN-IN"> </span>If you are not the intended
recipient, you are notified that disclosing, copying, distributing or taking any
action in reliance on the contents of this mail and attached file/s is strictly
prohibited<span lang="EN-IN">. Please notify the
sender immediately and delete this e-mail from your system. E-mail transmission
cannot be guaranteed to be secured or error-free as information could be
intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain
viruses. The sender therefore does not accept liability for any errors or
omissions in the contents of this message, which arise as a result of e-mail
transmission.</span></p>