Hi Mary,
thanks for your review :-). I was running older build and I run
./slim_test -c tests.nose

in directory usr/src/tools/tests.

I wasn't following the development for some time because I was working on some S10 bugs. So what is current recommended way of testing ? Was unittest replaced by something ?

Regards,

Tomas D.

Dne  9.05.12 19:02, Mary Ding napsal(a):
Tomas:

When you run your slim_install_unit test, what is the system running ??? Is it 
s11u1_15a ???

There is no more slim_install_unit test for quite a while in s11u1.  I am going 
to close out 7043709 since it did not fail any more for a while.

Other than that, your fix is fine.


----- Original Message -----
From: [email protected]
To: [email protected]
Sent: Wednesday, May 9, 2012 9:44:10 AM GMT -08:00 US/Canada Pacific
Subject: [caiman-discuss] code review request for 7161840

Hi all,
I would like to ask you for a code review for:

7161840 AI Sparc client fail to boot when AI service setup with
restrictive umask on S11 FCS

Webrev is here:

https://cr.opensolaris.org/action/browse/caiman/t.dzik/7161840/webrev/

Testing:
1) Sources are pep8 clean
2) I run unittest in gate and the only failed test was:
already logged bug 7043709 which has nothing to do with my fix
3) I reproduced the bug on sparc than built the gate on sparc, and
updated to built version. I run installadm create-service -s
/path/to/iso and verified that client machine is able to install.

Regards,

Tomas D.
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to