On Fri, Jul 27, 2012 at 03:57:16PM -0700, Dan Price wrote:
>
> I introduced a bug in our terminal handling when TERM isn't set or
> is set to a bogus value. Here's a fix:
>
> https://cr.opensolaris.org/action/browse/pkg/dp/setupterm/
This looks good. It took me a moment to realize that the exc
On Mon, Jun 18, 2012 at 01:15:04PM -0700, Danek Duvall wrote:
> Edward Pilatowicz wrote:
>
> >
> > https://cr.opensolaris.org/action/browse/pkg/edp/pkg.pli.cr3/webrev.cr1.diff
>
> nrlock.py:
>
> - line 58: docstring is identical to the one for locked(); probably could
> have a couple
On Tue, Apr 24, 2012 at 03:13:44PM -0700, Danek Duvall wrote:
> As a follow-up to Brock's fix for the gate not building, a full lint whack
> on the zoneproxy utilities:
>
> https://cr.opensolaris.org/action/browse/pkg/dduvall/zplint
Looks good.
-j
On Tue, Apr 24, 2012 at 11:54:48AM -0700, John Fischer wrote:
> On 04/24/12 11:38 AM, johan...@opensolaris.org wrote:
> >openssl s_client -CApath /etc/certs/CA -connect 192.168.100.10:4321
> depth=0 C = US, ST = California, L = PV, O = Oracle, OU = Solaris
> Install, CN =
> 192.168.100.10
> verify
On Tue, Apr 24, 2012 at 10:46:09AM -0700, John Fischer wrote:
> # pkg refresh
> Refreshing catalog 3/3
> pkg: 2/3 catalogs successfully updated:
>
> Framework error: code: 60 reason: SSL certificate problem, verify
> that the CA cert is OK.
> Details:
> error:14090086:SSL routines:SSL3_GET_SERVER_
On Wed, Mar 14, 2012 at 01:47:12PM -0700, Steve Sundstrom wrote:
> > So what happens when you check that the network is reachable before you
> > execute the operation, and then a network failure happens later during
> > the operation? You've just moved the point of failure without actually
> >
On Tue, Mar 13, 2012 at 12:59:41PM +1300, Tim Foster wrote:
> On the server-side, we'll add a new "proxied-urls" list property to
> each existing "publishers" entry returned by the publisher/0/
> response.
>
> The list contains a series of [uri, proxy] lists so that clients
> using -p can discover
On Wed, Mar 07, 2012 at 05:39:11PM +1300, Tim Foster wrote:
> Ok, I've had another go at this, and have a webrev at:
>
> https://cr.opensolaris.org/action/browse/pkg/timf/poisoned-cache-v3
Thanks for making these modifications. I like this change. Just a few
nits:
stats.py:
- line 320: typo
On Tue, Mar 06, 2012 at 11:55:42AM +1300, Tim Foster wrote:
> On 03/ 6/12 11:17 AM, johan...@opensolaris.org wrote:
> >On Tue, Mar 06, 2012 at 10:08:00AM +1300, Tim Foster wrote:
> >>I've got an incremental, and new webrev at:
> >>https://cr.opensolaris.org/action/browse/pkg/timf/poisoned-cache-v2/
On Tue, Mar 06, 2012 at 10:08:00AM +1300, Tim Foster wrote:
> I've got an incremental, and new webrev at:
> https://cr.opensolaris.org/action/browse/pkg/timf/poisoned-cache-v2/
On lines 1286-1298 and 1887-1899 you're not recording the fact that
you've seen a content error, which is going to skew t
Hi Ervin:
On Fri, Dec 23, 2011 at 01:33:33AM -0800, Ervin Yan wrote:
> Since our customers may only need 4 of the packages for basic
> functions, and need not download the whole image from our production
> server, and to provide consistent customer experiences with other
> platforms, we decided
Author: Krister Johansen
Repository: /hg/pkg/gate
Latest revision: 54d9bb03f34a7aed7c43f3ce5712e33fffbc4b56
Total changesets: 1
Log message:
19041 zoneproxyd door_server_create can race with door_create
Files:
update: src/zoneproxy/zoneproxyd/zoneproxyd.c
up confusing the code that binds the newly created thread to
the door's private pool. Long story short: more synchronization needed.
https://cr.opensolaris.org/action/browse/pkg/johansen/19041/webrev/
Thanks,
-j
___
pkg-discuss mailing
On Fri, Jun 10, 2011 at 08:39:02PM -0700, Brock Pytlik wrote:
> Webrev:
> http://cr.opensolaris.org/~bpytlik/ips-18463-v1/
The transport engine handles connect timeouts, speed timeouts, and
checks for libcurl stalls too. If you're trying to get something that's
robust as the rest of the data tran
On Fri, Jun 10, 2011 at 12:13:11AM -0700, Brock Pytlik wrote:
> 1) If pkg unfreeze doesn't match any frozen packages, no
> error is reported. I decided that seems reasonable since pkg
> unfreeze details each package that was unfrozen, so no output means
> no packages were unfrozen.
Not that you c
On Mon, Jun 06, 2011 at 02:05:14PM -0700, Brock Pytlik wrote:
> On 06/04/11 17:14, Edward Pilatowicz wrote:
> >http://mcescher.us.oracle.com/export/ws/pkg.perf2/webrev/
> >
> common.py:
> 2229: If there's any chance of the command being run generating a
> copious amount of error output, then a p.co
Author: johansen
Repository: /hg/pkg/gate
Latest revision: 8e786dd020035dedd71aaf134c11c3783a4a8026
Total changesets: 1
Log message:
18449 zoneproxyd dies while trying to set max number of file descriptors
Files:
update: src/zoneproxy/zoneproxyd/zoneproxyd.c
Folks,
I have a small fix to zoneproxyd (< 20 lines). Could somebody please
take a quick look at this one?
http://cr.opensolaris.org/~johansen/webrev-18449/
Thanks,
-j
___
pkg-discuss mailing list
pkg-discuss@opensolaris.org
h
On Mon, May 23, 2011 at 01:00:06PM -0700, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains changes for the following RFE:
>
> 18325 image-create should accept simple paths for sources
>
> webrev:
> http://cr.opensolaris.org/~swalker/pkg-18325/
This looks fine. I just have
On Tue, May 31, 2011 at 01:48:09PM -0700, Erik Trauschke wrote:
> On Mon, 2011-05-23 at 18:18 -0700, Erik Trauschke wrote:
> > I put out a webrev for
> > 18401 third party libraries should be moved to userland consolidation.
> >
> > http://cr.opensolaris.org/~erisch/pkg_gate0523/
>
> Could anyone
On Fri, May 27, 2011 at 11:30:59AM -0700, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains a fix for the following issue:
>
> 18438 temporary workaround needed for leaked fd after calls to
> libbe_py beList
>
> webrev:
> http://cr.opensolaris.org/~swalker/pkg-tip/
On Wed, May 25, 2011 at 10:49:48AM +1200, Tim Foster wrote:
> Sure. One other solution might be to have the zone-proxy-client
> instances simply do nothing in the face of a missing connection to the
> zone-proxyd (perhaps it could write a single SMF log entry to that
> effect, but leave the client
On Wed, May 25, 2011 at 10:38:11AM +1200, Tim Foster wrote:
> I agree. The problem here is, that at the moment if the sysrepo breaks,
> then zones-proxyd goes offline. Then all of the zones-proxy-client
> services will drop to maintenance and need to be manually cleared in
> each zone on the syst
On Mon, May 23, 2011 at 11:04:12PM -0700, Edward Pilatowicz wrote:
> i talked about this with krister last week and the conclusion we reached
> was that if the sysrepo port was changed and the service was refreshed,
> any outstanding connections would get dropped, and then to make sure new
> connec
On Tue, May 24, 2011 at 02:44:14PM +1200, Tim Foster wrote:
> On Mon, 2011-05-23 at 19:07 -0700, Edward Pilatowicz wrote:
> > why are we using restart_on='restart'?
>
> I think because zones-proxyd pulls configuration from the
> system-repository SMF service, in particular, which port to open
> co
On Mon, May 16, 2011 at 04:05:59PM -0700, Edward Pilatowicz wrote:
> 18342 pylint in the build is too noisy, it should only report errors
Does this mean that I won't get a notification that my code is rated
10/10 every time a build? I was enjoying that extra boost to my self
esteem. :(
-j
__
Author: johansen
Repository: /hg/pkg/gate
Latest revision: 4b4ff1617157c6c0dce485e90d88bc36021ecf7b
Total changesets: 1
Log message:
18277 zoneproxy client should release socket on orderly shutdown
18278 zoneproxy should excise list code
18279 zoneproxyd needs to refresh its config
18291
On Wed, May 11, 2011 at 12:02:18PM +1200, Tim Foster wrote:
> Hi Krister,
>
> On Tue, 2011-05-10 at 16:10 -0700, johan...@opensolaris.org wrote:
> > Folks,
> > Here's a webrev for some zoneproxy fixes that didn't make the cutoff for
> > b165.
> >
On Wed, May 11, 2011 at 09:03:23AM -0700, Danek Duvall wrote:
> johan...@opensolaris.org wrote:
>
> > http://cr.opensolaris.org/~johansen/webrev-18277/
>
> This isn't part of your change, but if someone could explain why we're
> setting LD_LIBRARY_PATH in
On Tue, May 10, 2011 at 05:40:42PM -0700, Edward Pilatowicz wrote:
> On Tue, May 10, 2011 at 04:10:26PM -0700, johan...@opensolaris.org wrote:
> > Folks,
> > Here's a webrev for some zoneproxy fixes that didn't make the cutoff for
> > b165.
> >
> >
Folks,
Here's a webrev for some zoneproxy fixes that didn't make the cutoff for
b165.
http://cr.opensolaris.org/~johansen/webrev-18277/
Thanks,
-K
___
pkg-discuss mailing list
pkg-discuss@opensolaris.org
http://mail.opensolaris.o
Author: johansen
Repository: /hg/pkg/gate
Latest revision: 63a4d56416c6942a1c1b0e9f0f7f726f88bb9b7c
Total changesets: 1
Log message:
18240 zone proxy needed
18265 pkg.sysrepo has weird notions about maximum port and cache size
Files:
create: src/brand/boot
create: src/brand/halt
On Tue, May 03, 2011 at 04:56:51PM -0700, Edward Pilatowicz wrote:
> -
> src/svc/zoneproxyd.xml
>
> - don't run as root. instead, run as daemon and add in just the privs
> you need. (which i'm guessing are file_owner and file_dac_read. if
> you need additional privs you can figure o
On Wed, Apr 20, 2011 at 02:35:26PM -0700, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains fixes for the following issues:
>
> 18193 transport can traceback for packages that belong to a publisher
> not in configuration
> 18195 transport adds all publisher caches fro
On Wed, Mar 30, 2011 at 09:36:05PM -0700, Brock Pytlik wrote:
> Here's the system repository work that Tim and I have been working
> on for quite a while now.
> http://cr.opensolaris.org/~bpytlik/ips-sysrepo-v1/
I took a look at the transport changes here, and they look fine.
-K
_
On Tue, Jan 04, 2011 at 04:09:22PM -0800, Shawn Walker wrote:
> Greetings,
>
> What follows is a simple change to setup.py to ensure our build
> works if mime types change unexpectedly for external sources:
>
> 17679 setup.py checks during extern source retrieval too strict
This looks right. I
On Fri, Dec 17, 2010 at 03:31:42PM -0800, Bart Smaalders wrote:
> One can certainly add per-version metadata to packages indicating
> that while version 12.1 is really 12-Weaselly Wombat!{1.2}g,
> version 12.2 is 12-Pernicious Platypus/#-+++0.3 q.
If it were up to me, I'd make 12.1 Alliteration Al
On Fri, Nov 19, 2010 at 02:43:22PM -0800, Danek Duvall wrote:
> Shawn Walker wrote:
>
> > http://cr.opensolaris.org/~swalker/pkg-tip/
>
> Looks okay to me.
And to me.
-j
___
pkg-discuss mailing list
pkg-discuss@opensolaris.org
http://mail.opensolari
On Fri, Nov 19, 2010 at 11:09:21AM -0800, Chris Quenelle wrote:
> Nothing's wrong if you know the right magic incantation.
> How is the user supposed to know that turning on "gdm" will give him a
> GUI login? I suppose Linux users would know because all Linux users
> are by definition system admin
On Fri, Nov 19, 2010 at 06:36:13PM +, Peter Dennis wrote:
> On 19/11/2010 18:23, johan...@opensolaris.org wrote:
> >If so, that's what the atime and mtime should be set to. Otherwise,
> >you're depending on implementation specific behavior. The code in
> >action.file makes a temporary file, a
On Fri, Nov 19, 2010 at 06:00:45PM +, Peter Dennis wrote:
> Hello,
>
> A question on timestamps when files are delivered to a system.
>
> o when doing the install the files receive timestamps based
> upon the time of the install:
>
> machine1:
> ls -l -% all ant
> -rwxr-xr-x 1 root
On Fri, Nov 12, 2010 at 02:59:32PM -0800, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains fixes for the following issue:
>
> 17398 repository rebuild index locking can cause traceback when using
> NFS
>
> webrev:
> http://cr.opensolaris.org/~swalker/pkg-nfs/
Looks
On Tue, Nov 09, 2010 at 05:32:07PM -0800, Shawn Walker wrote:
> On 11/ 9/10 05:25 PM, Tim Foster wrote:
> >On Tue, 2010-11-09 at 17:04 -0800, Shawn Walker wrote:
> >>http://cr.opensolaris.org/~swalker/pkg-tip/
> >
> >Yep, this looks good. pkglint does catch the errors, but only when
> >linting aga
On Tue, Nov 09, 2010 at 05:04:12PM -0800, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains fixes for the following issue:
>
> 17379 packages for pkg have wrong groups assigned to various lib/svc
> entries
>
> http://cr.opensolaris.org/~swalker/pkg-tip/
This looks rig
Author: johansen
Repository: /hg/pkg/gate
Latest revision: 796ccf4dfa896c3938d3d5d147aa4889c30dfba6
Total changesets: 1
Log message:
17377 content encoding errors should be retryable
Files:
update: src/modules/client/transport/exception.py
This one is so small that I'm attaching the diffs instead of generating
a webrev.
A user recently hit a problem with a gzip encoded download that kept
failing with an E_BAD_CONTENT_ENCODING error. It turns out that if the
gzip stream gets a CRC error, we can return this error. It should
be retry
Author: johansen
Repository: /hg/pkg/gate
Latest revision: c9985876d799e7f3b3b8ab92a58c131d2a2d0792
Total changesets: 1
Log message:
17327 pkg image-update to 151 raises NRLockException
Files:
update: src/modules/client/api.py
___
pkg-discuss
Folks,
Can I get a quick code review for the following bug fix? This is a
two-line change, so it ought to be an easy review.
http://cr.opensolaris.org/~johansen/webrev-17327/
Thanks,
-j
___
pkg-discuss mailing list
pkg-discuss@opensolaris.org
On Mon, Nov 08, 2010 at 07:29:37AM -0800, johan...@opensolaris.org wrote:
> On Fri, Nov 05, 2010 at 08:18:16PM -0700, Darren Reed wrote:
> > pkg: An unexpected error happened during image-update: Release of
> > unacquired lock
> >
> > Traceback (most recent call last):
> > File "/usr/bin/pkg", l
On Fri, Nov 05, 2010 at 08:18:16PM -0700, Darren Reed wrote:
> pkg: An unexpected error happened during image-update: Release of
> unacquired lock
>
> Traceback (most recent call last):
> File "/usr/bin/pkg", line 4242, in handle_errors
> __ret = func(*args, **kwargs)
> File "/usr/bin/pkg"
On Thu, Nov 04, 2010 at 02:09:27PM -0700, Shawn Walker wrote:
> On 11/ 4/10 01:21 PM, johan...@opensolaris.org wrote:
> >On Thu, Nov 04, 2010 at 11:33:21AM -0700, Shawn Walker wrote:
> >>Greetings,
> >>
> >>The following webrev contains fixes for the following issue:
> >>
> >> 17350 depot smf sor
On Thu, Nov 04, 2010 at 11:33:21AM -0700, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains fixes for the following issue:
>
> 17350 depot smf sort_file_max_size default of '' causes search
> indexing problems
>
> webrev:
> http://cr.opensolaris.org/~swalker/pkg-1735
On Thu, Oct 28, 2010 at 04:21:54PM -0700, Shawn Walker wrote:
> On 10/28/10 04:15 PM, Shawn Walker wrote:
> >On 10/28/10 04:02 PM, Danek Duvall wrote:
> >>Shawn Walker wrote:
> >>
> >>>http://cr.opensolaris.org/~swalker/pkg-15411/
>
> I've updated the webrev in place. Changed files have a comment
On Tue, Oct 26, 2010 at 03:32:17PM -0700, Danek Duvall wrote:
> Shawn Walker wrote:
>
> > On 10/26/10 02:31 PM, Danek Duvall wrote:
> > >Shawn Walker wrote:
> > >
> > >> http://cr.opensolaris.org/~swalker/pkg-tip/
> >
> > I've updated the webrev based on comments.
> >
> > There's no diff as I
On Mon, Oct 25, 2010 at 01:15:54PM +0100, Jon Anderson wrote:
> Hi Guys,
>
> Trying to update from 134 -> latest in http://ipkg.sfbay/dev which
> I think is 150. Getting this error:
>
> Expected 48353 files, received 48354 files instead.
>
> Expected 790177689 bytes, received 790179005 bytes ins
On Mon, Oct 18, 2010 at 02:33:17PM -0700, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains fixes and enhancements for pkgrecv and
> pkgsend:
>
> 16818 pkgrecv -d should publish per-package, not in one lump
> 9197 pkgrecv could show progress indicator for republishing
> 919
On Tue, Oct 12, 2010 at 02:36:30PM -0700, Shawn Walker wrote:
> Greetings,
>
> The webrev linked below contains changes for the following items:
>
> 7276 /var/pkg metadata needs reorg (looks busy)
> 5750 add versioning information for image meta data (cfg_cache, etc.)
> 7273 manifests shoul
On Tue, Oct 12, 2010 at 02:36:30PM -0700, Shawn Walker wrote:
> Greetings,
>
> The webrev linked below contains changes for the following items:
>
> 7276 /var/pkg metadata needs reorg (looks busy)
> 5750 add versioning information for image meta data (cfg_cache, etc.)
> 7273 manifests shoul
On Mon, Oct 04, 2010 at 03:34:06PM -0700, Bart Smaalders wrote:
> Pretty simple:
>
> http://cr.opensolaris.org/~barts/12530/
Looks good.
-j
___
pkg-discuss mailing list
pkg-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/pkg-discus
On Mon, Oct 04, 2010 at 02:00:02PM -0700, Brock Pytlik wrote:
> Webrev:
> http://cr.opensolaris.org/~bpytlik/ips-17151-v1/
>
> Bug:
> 17151 importer.py crashes post build 149
>
> The bug contains more details bug the short version is that the api
> version needed revving and the variants co
Author: johansen
Repository: /hg/pkg/gate
Latest revision: 6a366b063036fba80ed3c2df865b1c1dfdec5e56
Total changesets: 1
Log message:
17144 Unix socket support is defunct
17145 Add pycurl support for proxy username and password
Files:
delete: doc/system-repository.txt
delete: src
tches
Pycurl to better support proxy authentication.
http://cr.opensolaris.org/~johansen/webrev-17144/
Thanks,
-j
___
pkg-discuss mailing list
pkg-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
On Mon, Sep 27, 2010 at 04:51:18PM -0700, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains fixes for the following issues:
>
> 17055 image configuration should use pkg.config classes
> 8785 ImageConfig.read() should reset itself completely before reading
> 16109 imageconfi
On Mon, Sep 27, 2010 at 03:06:11PM +1300, Tim Foster wrote:
> Hi All,
>
> I've got a small pkglint webrev here, which fixes some usability bugs &
> enhancements, and was wondering if I could get a code review please?
>
> http://cr.opensolaris.org/~timf/pkglint-cli-webrev
lint/engine.py
- line
On Wed, Sep 22, 2010 at 03:39:45PM -0700, Shawn Walker wrote:
> On 09/10/10 06:48 PM, Shawn Walker wrote:
> >Greetings,
> >
> >The following webrev contains changes for the following items:
> >
> >2775 pkg should provide a way to downgrade (reinstall) packages
> >15462 pkg install subcommand needs
Hey Tim,
On Thu, Sep 23, 2010 at 09:22:29AM +1200, Tim Foster wrote:
> http://cr.opensolaris.org/~timf/info-classification-webrev
I asked Danek about this a couple of weeks ago, but his opinion was that
two reviewers were only necessary for our restricted builds, or when
additional scrutiny was d
On Fri, Sep 17, 2010 at 06:38:53PM -0700, Brock Pytlik wrote:
> On 09/17/10 06:24 PM, johan...@opensolaris.org wrote:
> >On Fri, Sep 17, 2010 at 06:13:41PM -0700, Brock Pytlik wrote:
> >> And yet one more webrev which addresses some issues with pkglint
> >>that Tim noticed.
> >>
> >>The major cha
On Fri, Sep 17, 2010 at 06:13:41PM -0700, Brock Pytlik wrote:
> And yet one more webrev which addresses some issues with pkglint
> that Tim noticed.
>
> The major changes are in the src/module/lint/* files. Also, a
> difference method has been added to VariantCombinationTemplate in
> src/modules/
On Wed, Sep 15, 2010 at 02:03:54PM -0700, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains a fix for the following issue:
>
> 17049 operations can fail with cache error when creating new be
>
> This was just introduced in the current gate build and I happened to
> spot the pr
On Tue, Sep 14, 2010 at 05:39:09PM -0700, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains fixes for the following issue:
>
> 17033 operations can fail with traceback if no packages are added
> or removed
>
> webrev:
> http://cr.opensolaris.org/~swalker/pkg-17033/
Looks good
On Tue, Sep 14, 2010 at 03:51:43PM -0700, Erik Trauschke wrote:
> Hi folks,
>
> so there seem to be a lot of folks who have questions about setting up
> Apache disk caching with the depot servers. So Shawn was thinking we
> should put an example in the gate:
>
> http://cr.opensolaris.org/~erisch/
Author: johansen
Repository: /hg/pkg/gate
Latest revision: 493e37f51c95bf9acb51d4db271e9b5dca42f5e9
Total changesets: 1
Log message:
17044 pkg(1) refers to non-existent pkg.mdnsd(1M) manpage
Files:
update: src/man/pkg.1.txt
___
pkg-discuss
I can't type today, which has made this whole bug, and fix, more
amusing. The following is a small fix for the pkg(1) manpage. The
mdns functionality is described in pkg.depotd(1m). The references to
pkg.mdnsd(1M) are for a manpage that we decided not to write during code
review, I just somehow
Hey Shawn,
On Fri, Sep 10, 2010 at 06:48:19PM -0700, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains changes for the following items:
>
> 2775 pkg should provide a way to downgrade (reinstall) packages
> 15462 pkg install subcommand needs to indicate it can be used for
> u
On Mon, Sep 13, 2010 at 03:05:12PM -0700, Shawn Walker wrote:
> On 09/ 9/10 03:45 PM, Shawn Walker wrote:
> >Greetings,
> >
> >The following webrev contains fixes for the following issues:
> >
> >16998 transport should support publisher-specific write and read caches
> >9572 FactoredManifest should
Author: johansen
Repository: /hg/pkg/gate
Latest revision: 604fffd79b385dd297c30ff40c6cde13945fa98a
Total changesets: 1
Log message:
16641 transport operations need ability to accept alternate Repository
Files:
update: src/modules/client/transport/transport.py
On Mon, Sep 13, 2010 at 01:37:43PM -0700, Danek Duvall wrote:
> johan...@opensolaris.org wrote:
>
> > http://cr.opensolaris.org/~johansen/webrev-16641/
>
> Sometimes you call it "alt_repository", and sometimes you call it
> "alt_repo". I take
ok at this, and is fine with the change.
If anyone else wants to review this before I put it back, let me know.
If I don't hear anything, I'll integrate later this afternoon.
http://cr.opensolaris.org/~johansen/webrev-16641/
Thanks,
-j
Hi Shawn,
On Thu, Sep 09, 2010 at 03:45:07PM -0700, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains fixes for the following issues:
>
> 16998 transport should support publisher-specific write and read caches
> 9572 FactoredManifest should allow consumers to specify cache l
On Thu, Sep 09, 2010 at 09:04:18PM -0700, Darren Reed wrote:
> >>I think I'll add a "pkg set publisher -d darrenr" to my
> >>first-boot-install service..
> >Maybe I don't understand your problem, but once you've disabled a
> >publisher it should stay disabled. In your script, please use --disable
On Thu, Sep 09, 2010 at 06:22:37PM -0700, Darren Reed wrote:
> On 9/09/10 05:48 PM, johan...@opensolaris.org wrote:
> >On Thu, Sep 09, 2010 at 05:40:46PM -0700, Darren Reed wrote:
> >>In my test environment, I ran a depot server as publisher "darrenr"
> >>to provide a custom package during the ini
On Thu, Sep 09, 2010 at 05:40:46PM -0700, Darren Reed wrote:
> In my test environment, I ran a depot server as publisher "darrenr"
> to provide a custom package during the initial install. I've since
> disabled that but now "pkg install" (to install a package from
> opensolaris.org) complains becau
On Thu, Sep 09, 2010 at 05:14:52PM -0700, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains changes for the following RFE:
>
>10148 successful image operations with no effect should be elided
> from history
>
> webrev:
> http://cr.opensolaris.org/~swalker/pkg-10148/
Looks g
On Thu, Sep 09, 2010 at 02:12:05PM -0700, Shawn Walker wrote:
> On 09/ 9/10 02:02 PM, johan...@opensolaris.org wrote:
> ...
> >IMO, the case for verifying compromised systems duplicates functionality
> >already implemented in bart(1) and tripwire, but I'm willing to listen
> >to reasonable argument
On Thu, Sep 09, 2010 at 01:46:34PM -0700, Darren Reed wrote:
> Should RFEs for pkg go into bugzilla or bugster?
>
> I think I can see two separate RFEs here:
>
> - being able to specify the source of the information used to
> verify the package manifests (and installed data) rather than
> rel
On Thu, Sep 09, 2010 at 09:19:38AM +0200, Paul van der Zwan wrote:
> Thanks, that fixed it, but this is the first time I ever needed to do
> this after extracting the repo isos IMHO That means something has
> changed in the way either the repo isos are built or the way the repo
> server behaves bec
On Wed, Sep 08, 2010 at 03:34:57PM -0500, Nicolas Williams wrote:
> (Avoiding the need to boot from media is in large part what Validated
> Execution is about.)
Read only root would have been a simpler solution to that problem.
-j
___
pkg-discuss mailin
On Wed, Sep 08, 2010 at 09:33:30AM +0200, Paul van der Zwan wrote:
> $ pkg search thunderbird
> pkg: Some repositories failed to respond appropriately:
> opensolaris.org:
> http protocol error: code: 503 reason: Service Unavailable
> URL: 'http://vetinari:8080/search/1/False_2_None_None_%3A%3A%3Ath
On Tue, Sep 07, 2010 at 02:28:57PM -0700, Ben Chang wrote:
>Thanks.
>
>Also thanks to those of you who answered Shawn's question about "make".
>
>BTW, are there any test suites that I need to run before RTI'ing
> my code change?
You might consider running pkglint(1) and pkgdep(1) aga
Hi Shawn,
On Sat, Sep 04, 2010 at 11:20:21PM -0700, Shawn Walker wrote:
> webrev:
> http://cr.opensolaris.org/~swalker/pkg-16982/
This looks good. Everything that follows are just nits.
man/pkg.depotd.1m.txt:
- lines 17-21: This is fine as is, but the editor in me wants to
break this in
ude
int
main(void)
{
int rc;
char *args[1];
args[0] = NULL;
rc = execv("/home/johansen/args", args);
if (rc < 0) {
perror("execv");
exit(EXIT_FAILURE);
}
return (0);
}
If I run the first
Did anybody review this fix? I would have liked to.
-j
On Tue, Sep 07, 2010 at 08:26:09PM +, shawn.wal...@oracle.com wrote:
> Author: Shawn Walker
> Repository: /hg/pkg/gate
> Latest revision: 2aef71869aaa5e3dda39f6b2371b7d9d990be6c3
> Total changesets: 1
> Log message:
> 16976 transport ca
Author: johansen
Repository: /hg/pkg/gate
Latest revision: a4a664a78547f8586a01a20b15635ecd419eaec3
Total changesets: 1
Log message:
16981 pkgrecv republication should get publisher prefix, not configuration
Files:
update: src/pull.py
___
pkg
On Fri, Sep 03, 2010 at 05:34:15PM -0700, Shawn Walker wrote:
> Seems right to me, although what about the lines around 457? We
> really want to use the source origins?
I had thought that the source publisher configuration would be less
problematic, but I'm willing to consider other options. My
Folks,
My most recent putback got the publisher configuration aspect of the
pkgrecv(1) republication wrong. The client should have been retrieving
the remote prefix, but not the entireity of the remote configuration.
This 2-line fix solves the problem, and makes everything work the way
that it sho
Author: johansen
Repository: /hg/pkg/gate
Latest revision: 874a83b3e8d10ad76999a5383218e47ada25a2c9
Total changesets: 1
Log message:
10390 make pkgrecv gracefully handle existing packages in destination
Files:
update: src/pull.py
update: src/tests/cli/t_pkgrecv.py
Hi Danek,
Thanks for reviewing this.
On Fri, Sep 03, 2010 at 02:13:19PM -0700, Danek Duvall wrote:
> johan...@opensolaris.org wrote:
>
> > http://cr.opensolaris.org/~johansen/webrev-10390/
>
> pull.py:
>
> - line 447: You might add a comment explaining w
On Fri, Sep 03, 2010 at 02:05:25PM -0700, Shawn Walker wrote:
> Yes, I'm aware it's simple to do, it just seemed somewhat wrong that
> given that a repository is a pre-defined set of directories that a
> creation operation wouldn't actually be the one to create that
> entire set.
Another possibili
On Fri, Sep 03, 2010 at 01:53:09PM -0700, Shawn Walker wrote:
> This was done intentionally because we essentially have no way to
> guarantee that the target directory isn't being used for something
> else or that continuing with the operation won't cause loss of data
> or other unexpected behaviou
On Thu, Sep 02, 2010 at 08:04:07PM -0700, Brock Pytlik wrote:
> Webrev:
> http://cr.opensolaris.org/~bpytlik/ips-16977-v1/
>
> Bug:
> 16977 Test needed for advanced signature version
This looks fine.
-j
___
pkg-discuss mailing list
pkg-discuss@opensola
1 - 100 of 1637 matches
Mail list logo
