Re: [Fuego] ltp_process.py is broken - anyone want to fix it?

Thanks!
 -- Tim


> -----Original Message-----
> From: [email protected]
> 
> Hi Tim,
> 
> I wrote it and we have fixed it internally but the person was busy and didn't upstream it.
> I will look into it.
> 
> Regards,
> Daniel
> 
> > -----Original Message-----
> > From: Fuego <[email protected]> On Behalf Of Bird, Tim
> > Sent: Friday, October 2, 2020 7:50 AM
> > To: [email protected]
> > Subject: [Fuego] ltp_process.py is broken - anyone want to fix it?
> >
> > ltp_process.py in test Functional.LTP is broken.
> >
> > Here's what I'm getting if I try to use it:
> >
> >   python error:
> >   from openpyxl.style import Border, Borders, Fill, Color, Alignment
> >   ImportError: No module named style
> >
> > I'm not sure when, but it appears that the openpyxl project refactored their code some time recently (how recently I don't know).  We
> > install the python-openpyxl module in the Fuego Dockerfile, and have for some time.
> > The current version we're getting is python-openpyxl 2.3.0-3.  I think in jessie the version 1.7.0.
> >
> > In any event, I dug into this bug didn't get very far.
> >
> > I think it just needs to have the imports corrected, but it might be deeper than this.
> >
> > Does anyone remember who wrote this?  I looked in the git  log, and Daniel has done some patches to this, but I'm not sure if he's the
> > original author.
> > Is it from Toshiba, or Fujitsu, or someone else?
> >
> > Anyway - if someone would like to take a look at this it would be great.
> >
> > I have temporarily disabled this by commenting out the call to it in fuego-core/tests/Functional.LTP/fuego_test.sh.  That means that
> > the links to the xls files are bad for LTP tests currently.  I didn't want to wholesale remove the feature before asking for help fixing it, as
> > it will disrupt the spec file pretty badly.
> >
> > Let me know if anyone knows about this and/or is willing to work on it.
> >
> > Thanks,
> >  -- Tim
> >
> > _______________________________________________
> > Fuego mailing list
> > [email protected]
> > https://lists.linuxfoundation.org/mailman/listinfo/fuego