ReconfigureIO / reco

A command line tool for interacting with Reconfigure.io services
Apache License 2.0
6 stars 3 forks source link

if there's a timeout event report it to user #5

Closed CampGareth closed 6 years ago

CampGareth commented 6 years ago

This should cause builds to show status timeout instead of error when a timeout has occurred. It does this by looking at the code field of the events of a job.

CampGareth commented 6 years ago

Tested on timed-out sims, works there, should work everywhere else due to being added to the base job. All dependencies are merged, so this is ready to merge

CampGareth commented 6 years ago

Let users know about sim and build timeouts

robtaylor commented 6 years ago

whats a good way to test this?

CampGareth commented 6 years ago

I've been using simulations with an infinite loop somewhere in them. e.g. in the addition example:

// Print the value we got from the FPGA
    for {
        fmt.Printf("%d\n", ret)
    }

Run a simulation of it and wait (1 hour timeout for simulations), then run reco test list This has been giving me output like this:

421a623c-bc7b-4829-8f1c-44294f533ac8      timed-out      3 days ago       1h0m11s   
  0cbe0195-4c7e-47d0-8e3a-4cbe7b3d8cb1      completed      3 days ago       18m43s    
  84b291cf-3a38-4b0d-9afc-6cb1708feef2      timed-out      3 days ago       1h0m31s   
CampGareth commented 6 years ago

Reco v0.4.0