[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 Dan Horák changed: What|Removed |Added Status|ASSIGNED|CLOSED

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #39 from Dan Horák --- Any tagging is not necessary, the build in ppc koji correctly landed in f24 tag, because bodhi is not in action yet for f24. -- You are receiving this mail because: You are on the CC list

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #38 from Vasant Hegde --- > Dan, > > I've imported package and built it for F24 branch [1]. Can you please tag > this to fedora 24? > > http://ppc.koji.fedoraproject.org/koji/taskinfo?taskID=3172026 I

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #37 from Vasant Hegde --- Dan, I've imported package and built it for F24 branch [1]. Can you please tag this to fedora 24? http://ppc.koji.fedoraproject.org/koji/taskinfo?taskID=3172026 -Vasant -- You

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #36 from Vasant Hegde --- (In reply to Jon Ciesla from comment #35) > Package request has been approved: > https://admin.fedoraproject.org/pkgdb/package/opal-prd Thanks! I'm able to import initial package

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #35 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/opal-prd -- You are receiving this mail because: You are on the CC list for the bug. You are always

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 Vasant Hegde changed: What|Removed |Added Blocks||1312018 --

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #34 from Vasant Hegde --- (In reply to Dan Horák from comment #33) > thanks, the package is APPROVED Thanks Dan. I will raise "SCM admin request" to include this package. -Vasant -- You are receiving

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 Dan Horák changed: What|Removed |Added Flags|fedora-review? |fedora-review+ ---

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #32 from Vasant Hegde --- V7: Changes in v7: Fix opal-prd recompilation issse during install Spec URL : https://hegdevasant.fedorapeople.org/opal-prd/v7/opal-prd.spec SRPM :

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #31 from Vasant Hegde --- Dan, > looks good, but I have found that opal-prd is being recompiled in the > %install section > > from the build.log > ... > Executing(%install): /bin/sh -e

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #30 from Vasant Hegde --- (In reply to Dan Horák from comment #28) > (In reply to Vasant Hegde from comment #27) > > Dan, > > > > > > > > The systemd guideline links to > > >

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #29 from Dan Horák --- (In reply to Vasant Hegde from comment #26) > V6: > > Chnages: > Added "Requires(post|preun|postun) tags" > > Spec URL : https://hegdevasant.fedorapeople.org/opal-prd/v6/opal-prd.spec > SRPM

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #28 from Dan Horák --- (In reply to Vasant Hegde from comment #27) > Dan, > > > > > The systemd guideline links to > > https://fedoraproject.org/wiki/Packaging: > > DefaultServices?rd=Starting_services_by_default

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #27 from Vasant Hegde --- Dan, > > The systemd guideline links to > https://fedoraproject.org/wiki/Packaging: > DefaultServices?rd=Starting_services_by_default for how to handle default > state of

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #26 from Vasant Hegde --- V6: Chnages: Added "Requires(post|preun|postun) tags" Spec URL : https://hegdevasant.fedorapeople.org/opal-prd/v6/opal-prd.spec SRPM :

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #25 from Vasant Hegde --- Dan, > > > - my scratch build = > > > http://ppc.koji.fedoraproject.org/koji/taskinfo?taskID=3077491 > > > - please update the scriptlets and Requires to the current style, see >

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #24 from Dan Horák --- (In reply to Vasant Hegde from comment #23) > V5: > - Rebased to latest upstream version (v5.1.13) > - Updated specfile to include pflash and scom tools in opal-utils package > - Updated

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 Hanns-Joachim Uhl changed: What|Removed |Added Blocks||1051573

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 Vasant Hegde changed: What|Removed |Added Flags|needinfo?(hegdevas...@in.ib |

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #22 from IBM Bug Proxy --- --- Comment From hegdevas...@in.ibm.com 2016-01-22 05:02 EDT--- (In reply to comment #5) > sorry for the delay, but formal review is here, see the notes explaining OK* > and

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #21 from Dan Horák --- sorry for the delay, but formal review is here, see the notes explaining OK* and BAD statuses below: OK source files match upstream: 080a0992dc4241ac0c8c2b7e556a20bbb45d068d

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 Hanns-Joachim Uhl changed: What|Removed |Added CC|

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #19 from Vasant Hegde --- (In reply to Dan Horák from comment #17) > one more question - is the separation between opal-prd and opal-utils > necessary? opal-prd contains recovery and diagnostics daemon..

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #20 from Vasant Hegde --- V4: Addressed review comments from Dan. Spec URL : https://hegdevasant.fedorapeople.org/opal-prd/v4/opal-prd.spec SRPM :

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #18 from Vasant Hegde --- Dan, Thanks for the review. (In reply to Dan Horák from comment #16) > - distro-wide CFLAGS are not honoured in the build - >

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 Hanns-Joachim Uhl changed: What|Removed |Added Blocks||1224121

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #12 from Vasant Hegde --- Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed Issues: === - Package installs properly.

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 Vasant Hegde changed: What|Removed |Added Blocks|1224121 | --- Comment

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #14 from Vasant Hegde --- v3: Spec URL : https://hegdevasant.fedorapeople.org/opal-prd/v3/opal-prd.spec SRPM : https://hegdevasant.fedorapeople.org/opal-prd/v3/opal-prd-5.1.11-3.fc22.src.rpm -Vasant --

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #15 from Vasant Hegde --- (In reply to Vasant Hegde from comment #14) > v3: > > Spec URL : https://hegdevasant.fedorapeople.org/opal-prd/v3/opal-prd.spec > SRPM : >

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 Hanns-Joachim Uhl changed: What|Removed |Added Blocks||1224121

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 Dan Horák changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #17 from Dan Horák --- one more question - is the separation between opal-prd and opal-utils necessary? Won't be these 2 packages always both installed? Or is the daemon accessible remotely? -- You are receiving

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #1 from Vasant Hegde --- I will build fedora package and post link here later this week. -Vasant -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #7 from Vasant Hegde --- Dan, (In reply to Dan Horák from comment #5) > %define kversion `uname -r` > won't work in the buildsystem (or even just in mock), the kernel running is > different from the

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #8 from Vasant Hegde --- v2: In v1, I missed to add systemd as Build Requirement. Hence build failed in mock environment. I've fixed that issue and now I'm able to build rpm in mock build environment.

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #9 from Vasant Hegde --- Koji build : http://ppc.koji.fedoraproject.org/koji/taskinfo?taskID=2930041 -Vasant -- You are receiving this mail because: You are on the CC list for the bug. You are always

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #10 from Vasant Hegde --- V2 updated: Spec URL: https://hegdevasant.fedorapeople.org/opal-prd/v2/opal-prd.spec SRPM: https://hegdevasant.fedorapeople.org/opal-prd/v2/opal-prd-5.1.11-2.fc22.src.rpm

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #11 from Vasant Hegde --- I ran fedora-review tool on this bugzilla and here is the output : cat rpmlint.txt Checking: opal-prd-5.1.11-2.fc22.ppc64.rpm opal-utils-5.1.11-2.fc22.ppc64.rpm

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 Dan Horák changed: What|Removed |Added CC||d...@danny.cz

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #4 from Vasant Hegde --- rpmlint output : rpmlint -v opal-prd-5.1.11-1.fc22.src.rpm opal-prd.src: I: checking opal-prd.src: I: checking-url http://github.com/open-power/skiboot (timeout 10 seconds)

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #3 from Vasant Hegde --- Spec URL: https://hegdevasant.fedorapeople.org/opal-prd/v1/opal-prd.spec SRPM URL: https://hegdevasant.fedorapeople.org/opal-prd/v1/opal-prd-5.1.11-1.fc22.src.rpm -Vasant -- You

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #5 from Dan Horák --- %define kversion `uname -r` won't work in the buildsystem (or even just in mock), the kernel running is different from the installed kernel-devel package, I think KERNEL_DIR=/usr/src/kernels/*

[Bug 1284527] Review Request: opal-prd - OPAL Processor Recovery Diagnostics daemon

https://bugzilla.redhat.com/show_bug.cgi?id=1284527 --- Comment #6 from Vasant Hegde --- Dan, (In reply to Dan Horák from comment #5) > %define kversion `uname -r` > won't work in the buildsystem (or even just in mock), the kernel running is > different from the