The graph hash used in the graphoid unit test has been updated and deployed.

On Wed, Dec 16, 2015 at 2:42 PM, Marko Obrovac <mobrovac@wikimedia.org> wrote:
On 16 December 2015 at 09:34, Giuseppe Lavagetto <glavagetto@wikimedia.org> wrote:
On Wed, Dec 16, 2015 at 1:24 AM, Yuri Astrakhan
<yastrakhan@wikimedia.org> wrote:
> Upon further digging, this might actually be expected:
> * The hash calculation has not changed

Indeed, the hash calculation hasn't changed, but you changed the graph defs, which altered the hash~[1], thus leading to monitoring failures for RESTBase. I suspect the same thing happened with the specification tests for Graphoid.

Marko
 
> * I manually updated many graphs recently, by changing the Lua code - about
> 25k graphs, which could be the reason why so many cached HTML pages were
> referring to older version of the images. The image URL contains a hash, but
> that hash is no longer present in the SQL (pageprops), hence the failure.
> * A semi-related bug was discovered - several non-wikipedia wikis that allow
> "Graph" namespace do not save graph data to the prop-pages sql table.  Since
> graph namespace is not used much, I think I will disable this support
> entirely, rather than try to figure it out.
>

So, when will you release a new version of graphoid which doesn't
expose a spec that fails to be verified?

Cheers,

Giuseppe


[1] https://en.wikipedia.org/w/index.php?title=User:Pchelolo/Graph&action=history

--
Marko Obrovac, PhD
Senior Services Engineer
Wikimedia Foundation