Lars - I downloaded your latest branch (7091) and ran testing (hpitest 3.1.1) on the dynamic simulator.
I'm getting 8 failures out of 711 test cases - using hpitest-3.1.1. A couple of comments: First - thanks for all the work. Second, in the openhpi.conf file - you still refer to the simulator as the new_simulator in your comments for your stanza. Might want to change that. Secondly, it does not appear that the IDR area count issue has been fixed. This is the issue that Mohan had described earlier. The area count and the number of actual IDR areas should always match. If you start up your dynamic_simulator plugin, then run /usr/bin/hpigensimdata and capture the output. The run the test: ./hpitest Inventory/saHpiIdrInfoGet/UpdateCounter Then run /usr/bin/hpigensimdata again, and compare the output to the first run of hpigensimdata - you will see the discrepancy. In second output of hpigensimdata, the area count is set at 2 - but the actual number of areas present in the data file is only 1. /usr/bin/hpigensimdata > mdata1.txt ./hpitest Inventory/saHpiIdrInfoGet/UpdateCounter /usr/bin/hpigensimdata > mdata2.txt diff mdata1.txt mdata2.txt 1715c1715 < UpdateCount=0 --- > UpdateCount=2 1717c1717 < NumAreas=1 --- > NumAreas=2 --michael Regards, Michael Bishop Enterprise Servers and Storage (ISB Linux/Telco) Hewlett-Packard Company 3404 E. Harmony Rd. Bldg. 5L, Post B7, Mailstop 42 Fort Collins, CO 80528-9599 Phone: 970-898-4393 E-Mail: [email protected] > -----Original Message----- > From: Lars Wetzel [mailto:[email protected]] > Sent: Thursday, June 03, 2010 2:52 PM > To: [email protected] > Cc: Bishop, Michael (ISB Linux/Telco); Devarajulu, Mohanasundaram; > Kleber, Ulrich (NSN - DE/Munich) > Subject: Re: [Openhpi-devel] New simulator testing > > > Hi Michael, Mohan and Uli, > > ok following changes were made: > > 1. and 3. debug and logfile > The logfile generation is now deactivted by default. > > 4. ep of announcements > Fix of simulation.data.example > > 6. naming > Change the naming to dynamic_simulator (library libdyn_simulator) > It works on my machine, but due to some problems with svn (I > had to commit in > 3 steps) I hope it works also with the data inside the repository. > Furthermore it is and was the most critical part of the changes. > > 2. new <-> old > it's still your part ;-) > > 5. timestamp of announcement > fault or not fault, it has to be discussed if it is really > necessary to work > with original timestamp in the announcements. > > soa_plugin: > Testcases > Sensor/saHpiSensorReadingGet/GetAndVerify > Sensor/saHpiSensorThresholdsGet/GetThresholds > Will now pass. > The reason was a inconsistence of the data. I enhanced the > hpigensimdata > client to overcome it. (DataFormat.IsSupported was set on > SAHPI_TRUE but > Reading.IsSupported returns SAHPI_FALSE) > I don't understand how the test cases can pass in your > system. Maybe it was a > temporarily bug? But nevertheless the client will throw a > warning if it > happens again. > > Sensor/saHpiSensorEventMasksSet/SensorEnableChangeEvent > Wasn't supported by old and new simulator. > Now it is supported in the new simulator and had a small bug. - Fixed > > Inventory/saHpiIdrAreaHeaderGet/TraversalByAreaType > Was a bug -> fixed > > Inventory/saHpiIdrFieldSet/SetField > {COOLING_DEVICE,x} and FANs > Fails due to wrong data -> Field has wrong Type entry > Can you please verify it with original hardware > > Inventory/saHpiIdrAreaAddById/AddAreaById > SAHPI_FIRST_ENTRY is a valid AreaId and can be used with this > function, > provided the IDR does not have a pre-existing Area with that AreaId. > In the data the areas starts with the Id 0 == > SAHPI_FIRST_ENTRY. So I think > the issue is more a problem of the test case?! > > HotSwap/saHpiAutoExtractTimeoutGet/GetTimeout > Was a Typo, is now fixed > > atca_data: > Sorry is still open: > POWER_CYCLE -> Event generation is missing > Textbuffer doesn't support binary data > > Hope I didn't bring in some new faults. > > Best Regards > Lars ------------------------------------------------------------------------------ ThinkGeek and WIRED's GeekDad team up for the Ultimate GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the lucky parental unit. See the prize list and enter to win: http://p.sf.net/sfu/thinkgeek-promo _______________________________________________ Openhpi-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openhpi-devel
