gmane.os.solaris.opensolaris.pkg.general

Code review reminder: pkg/update fix and sysrepoconfig resiliency

Tim Foster — 2013-03-19T20:51:25

Hi there,

Just a quick reminder that I've got two small fixes out for code review 
at the moment, and was wondering if anyone had time to take a look?

The webrevs are:

https://cr.opensolaris.org/action/browse/pkg/timf/sysrepo-resilient-config/sysrepo-resilient-config-webrev/

https://cr.opensolaris.org/action/browse/pkg/timf/add-cronjob/add_cronjob-webrev

The original mails to pkg-discuss were:

Subject: Code review: making pkg.sysrepo more resilient in the face of 
inaccessible file repos
  (sent 02/26)

Subject: Code review: add_cronjob failing after unclean shutdown
  (sent 03/14)

cheers,
tim

p5p package install with parent dependency withinlocal zones

Sajith C R — 2013-03-19T11:54:41

Hi IPS team,
I am facing following issue with p5p archive package install inside local
zones on solaris 11.
I use p5p format package to install my package on solaris 11 and the
package contains following entries in the manifest to define parent
dependency.

depend type=parent fmri=feature/package/dependency/self
variant.opensolaris.zone=nonglobal

Prior to solaris 11 update, since the file based p5p publisher is not
propagated inside a zone automatically, I had to install the p5p package
inside local zones by copying the p5p file inside zone and then setting
publisher inside zone which was working fine for me.

with parent dependency mentioned, the package upgrades are not working with
my p5p format packages.
When publisher is set at global zone, IPS complain about version mismatch
with local zone.
If global zone is upgraded after detaching the local zone, later zone
attach fails because of package version mismatch.

I want to how to upgrade such packages on a pre-solaris 11 update1 setups?

With solaris 11u

How to repair /etc/pam.conf

James Litchfield — 2013-03-18T19:03:12

Through some mysterious process, I have ended up with a
zero length /etc/pam.conf.

The intriguing thing is how do I rectify this situation?

I can find the file in the package, I can do a pkg fix on the
package that contains it and I still end up with what I had before.
One possibility is that it gets built on the fly?

This does bring up a question: how does one extract a
single file from a package in a supported manner?

Jim
----

Code review: add_cronjob failing after uncleanshutdown

Tim Foster — 2013-03-14T05:56:13

hi there,

I've a short webrev here that fixes the case where a system was rebooted 
without the SMF stop-method script being called for the services that 
use the pkg5_include.sh function 'add_cronjob', causing the subsequent 
start-method script to fail.

This bug affects svc:/application/pkg/update and the new 
svc:/application/pkg/mirror services.

https://cr.opensolaris.org/action/browse/pkg/timf/add-cronjob/add_cronjob-webrev

Comments welcome,

cheers,
tim

safety of the update (downgrade) operation

Joe Higgins — 2013-03-12T13:29:20

Hi,

I had a general question about the pkg update operation.

With the update operation the user can pick specify a version which 
might actually be a downgrade on a installed package.

I see one caveat in the man page about

Please note that updating
           specific packages across package rename or obsolete boundaries
           is not supported.


So are they any additional scenarios WRT to downgrading packages which 
customers need to be concerned about.


Joe

15804991 option mangling and verification should bepart of the API, not the CLI client

2013-03-11T22:34:23

Author: Erik Trauschke <Erik.Trauschke< at >oracle.com>
Repository: /hg/pkg/gate
Branch: default
Latest revision: 09e276ba70c65d321c8d9f631b70bf74be5fc01d
Total changesets: 1
Log message:
15804991 option mangling and verification should be part of the API, not the CLI client

Files:
create: src/modules/client/options.py
update: src/client.py
update: src/modules/client/api_errors.py
update: src/modules/misc.py
update: src/pkg/manifests/package:pkg.p5m
update: src/po/POTFILES.in

IPS project migrating to java.net

Danek Duvall — 2013-03-08T22:11:08

In an effort to consolidate our support infrastructure, we are moving the
remaining projects on opensolaris.org to java.net, including IPS.  The
transition will be complete on March 24, as you may have noticed.

The project on java.net will be available via the following URL:

    http://java.net/projects/ips

The source code is available from the following URLs:

    ssh://<username>< at >hg.java.net/ips~pkg-gate
    https://hg.java.net/hg/ips~pkg-gate

The latter URL does not require any association with the project or even
with java.net -- you can retrieve the source without registering.

You will need to register with java.net to stay on the mailing list,
though, which is now

    pkg-discuss< at >ips.java.net

We can't transfer user accounts automatically, due to privacy restrictions
and the fact that usernames might conflict, so you'll have to register and
subscribe yourself.

As of this time, java.net does not provide a dedicated mechanism to post
code reviews, so we're piggy-backing on the "website" feature, t

Added tag s12b17 for changeset 9339dcdbcf9d

2013-03-08T18:07:14

Author: Danek Duvall <danek.duvall< at >oracle.com>
Repository: /hg/pkg/gate
Branch: default
Latest revision: 5a3a050c4d978f1bd30e3f1582379561d224f145
Total changesets: 1
Log message:
Added tag s12b17 for changeset 9339dcdbcf9d

Files:
update: .hgtags

16447594 resync nroff manpages from docs group

2013-03-06T23:20:21

Author: Danek Duvall <danek.duvall< at >oracle.com>
Repository: /hg/pkg/gate
Branch: default
Latest revision: 9339dcdbcf9db7b0e773609c53ddadfb191f879c
Total changesets: 1
Log message:
16447594 resync nroff manpages from docs group

Files:
update: src/man/packagemanager.1
update: src/man/pkg.1
update: src/man/pkg.5
update: src/man/pkg.depot-config.1m
update: src/man/pkg.depotd.1m
update: src/man/pkg.sysrepo.1m
update: src/man/pkgdepend.1
update: src/man/pkgdiff.1
update: src/man/pkgmerge.1
update: src/man/pkgmogrify.1
update: src/man/pkgrecv.1
update: src/man/pkgrepo.1
update: src/man/pkgsend.1
update: src/man/pm-updatemanager.1

code review request: man page updates

Danek Duvall — 2013-03-06T22:31:09

    https://cr.opensolaris.org/action/browse/pkg/dduvall/pkg-nroff/

These are pulled from the man page gate, and cover

    14851656 pkg.5: clarify use of user and group actions
    15813906 pkgrecv man page should note parallel pkgrecv is unsupported
    15822251 pkgrecv(1) and pkgsend(1) need to mention use of pkgrepo to bu
    15822468 pkgmerge.1 needs to document the -p option
    15954795 pkg.1: incorrect use of ellipses for --reject argument
    16032473 pkgrepo(1) should document verify and fix subcommands
    16273705 need a man page for pkg.depot-config
    16319221 man pages for pkgrecv, pkgsend and pkgrepo need to document ssl
    16390733 pkg.1 update description of pkg revert --tagged tag-name
    16390837 pkg(1) should expound on the information included in history entries
    16390874 pkg.1 clarify the description of pkg image-create
    16404227 pkgdepend(1) changes for 7140660 have gone missing
    16404765 pkgdiff.1 update to add stdin input and and -t option
    16404854 pkg.5 add revert-

Need to replace Sun Connection Update with IPS

Jayesh K Jain — 2013-03-06T12:31:53

Hi Team,

I would like to support End Point Management with Solaris 11. I have couple
of questions on the same .


1. How should I register my endpoints (Solaris 11 system) with new IPS
system??

2.What all perquisites needed to  execute the command "pkg update" from end
points.

3.For Solaris 10 patch management we were using getupdates1.sun.com but
after IPS implement we would require some new computer name or host name or
repository name
    so could you please provide the same with all details like
protocol,port number etc.

4.Earlier for registration of endpoints with Solaris 10  we were using "
/usr/sbin/sconadm register -a -r /tmp/RegistrationProfile.properties  .
   now How this would be implement by IPS to support Solaris 11.


Regards
Jayesh Jain

16419678 pkg/mirror should empty its cache on success

2013-03-05T21:54:32

Author: Tim Foster <tim.s.foster< at >oracle.com>
Repository: /hg/pkg/gate
Branch: default
Latest revision: ea17c3768631d581ea0ba6c4dab664ea33f8ed61
Total changesets: 1
Log message:
16419678 pkg/mirror should empty its cache on success
16427874 pkg/mirror should set the default publisher if not already set

Files:
update: src/svc/svc-pkg-mirror

pkg install to an alternative directory

Jeffrey J Huang — 2013-03-05T13:34:59

Hi, Pkg Experts,

I want know for "pkg install", does it have the similar function as
'pkgadd -R'?

For SVR4 package, 'pkgadd -R <dir>' allows to choose a different
root_path other
than '/', and the <dir> could almost be anywhere like "/tmp".

For pkg, although there is '-R <dir>', but it requires a BE there before
install.
But I don't have image rooted there or I could not create a BE when I am
in Solaris Mirco-root state.

So is there any workaround for this to let "pkg install" to install
packages other
base dir than root?

Great Thanks
Jeffrey

Code review: two small pkg/mirror fixes

Tim Foster — 2013-03-03T23:33:49

Hi there,

I've a small webrev here for the pkg/mirror service to delete the 
pkgrecv cache on success, and makes sure we set 'publisher/prefix' on 
the repository we create (if it's not already set)

https://cr.opensolaris.org/action/browse/pkg/timf/mirror-cache/mirror-cache-webrev/

Comments would be most welcome,

cheers,
tim

14851733 pkgrecv should default to retrevingall-timestamps not justlatest versions

2013-03-01T23:00:38

Author: Erik Trauschke <Erik.Trauschke< at >oracle.com>
Repository: /hg/pkg/gate
Branch: default
Latest revision: 5132092a44363564e80fc5a0a94896a8b68161c5
Total changesets: 1
Log message:
14851733 pkgrecv should default to retreving all-timestamps not just latest versions

Files:
update: src/man/pkgrecv.1
update: src/pull.py
update: src/tests/cli/t_pkgrecv.py

Added tag s12b16 for changeset 1ec958dbd40a

2013-03-01T19:24:29

Author: Danek Duvall <danek.duvall< at >oracle.com>
Repository: /hg/pkg/gate
Branch: default
Latest revision: 772ef1b0cb9db20343ad1f405dda1382a6b1851c
Total changesets: 1
Log message:
Added tag s12b16 for changeset 1ec958dbd40a

Files:
update: .hgtags

[review] factoredmanifest caching fix

Shawn Walker — 2013-03-01T01:02:33

Greetings,

The following webrev contains fixes for the following issue:

   16389464 multiple whitespaces in action path can cause package
            metadata errors

https://cr.opensolaris.org/action/browse/pkg/swalker/pkg-16389464/webrev

-Shawn

16390889 need test for original_name preservation forfile to link

2013-03-01T00:57:11

Author: Shawn Walker <shawn.walker< at >oracle.com>
Repository: /hg/pkg/gate
Branch: default
Latest revision: 1a53c1fce591f1bc31aed6bfe2763aeba8612f99
Total changesets: 1
Log message:
16390889 need test for original_name preservation for file to link

Files:
update: src/tests/cli/t_pkg_install.py

16412707 pkg/depot method script misusing ppriv

2013-02-28T04:20:22

Author: Tim Foster <tim.s.foster< at >oracle.com>
Repository: /hg/pkg/gate
Branch: default
Latest revision: 143fa24eb3469d2afa1e787e8d620ff0b9df4e75
Total changesets: 1
Log message:
16412707 pkg/depot method script misusing ppriv

Files:
update: src/svc/svc-pkg-depot

16069417 need a service to periodically update apkg(5) repository

2013-02-28T03:26:28

Author: Tim Foster <tim.s.foster< at >oracle.com>
Repository: /hg/pkg/gate
Branch: default
Latest revision: 793902704e218cfe4e65a04276e08431615ca9e8
Total changesets: 1
Log message:
16069417 need a service to periodically update a pkg(5) repository

Files:
create: src/svc/pkg-mirror.xml
create: src/svc/pkg-repositories-setup.xml
create: src/svc/pkg5_include.sh
create: src/svc/svc-pkg-mirror
create: src/svc/svc-pkg-repositories-setup
create: src/svc/svc-pkg-update
create: src/util/misc/user_attr.d/package:pkg
delete: src/um/pkg-update
update: src/pkg/manifests/package:pkg.p5m
update: src/pkg/manifests/package:pkg:update-manager.p5m
update: src/pkg/transforms/defaults
update: src/setup.py
update: src/svc/pkg-update.xml
update: src/util/misc/auth_attr.d/package:pkg

Code review: ppriv typo in svc-pkg-depot

Tim Foster — 2013-02-28T03:02:44

Hi there,

I've got a small code review for a mistake I made when putting back 
16292464, where I forgot to add the "$$" argument when calling ppriv in 
the svc:/application/pkg/depot method script.

I've filed this as:
16412707 pkg/depot method script misusing ppriv

The fix is small enough that I've just inlined it below.

Apologies for not catching this sooner.

cheers,
tim


timf< at >kura[448] hg diff
diff --git a/src/svc/svc-pkg-depot b/src/svc/svc-pkg-depot
--- a/src/svc/svc-pkg-depot
+++ b/src/svc/svc-pkg-depot
< at >< at > -226,7 +226,7 < at >< at >
  cmd="start"
  run_depot
  # drop privileges now that we've written our configuration
-/usr/bin/ppriv -s E=basic,net_privaddr
+/usr/bin/ppriv -s E=basic,net_privaddr $$
  run_htcacheclean
          emsg=$(/usr/bin/printf ${FAILED_TO_RUN} start)
  ${HTTPD} -f ${depot_runtime_dir}/depot_httpd.conf \
< at >< at > -238,7 +238,7 < at >< at >
  cmd="graceful"
  run_depot
  # drop privileges now that we've written our configuration
-/usr/bin/ppriv -s E=basic,net_privaddr
+/usr/bin/ppriv -