-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Wed, 11 Feb 2009 11:20:31 -0500
Jason Dobies <[email protected]> wrote:

> Looks like this class hasn't really been changed since it was moved to
> git, so I figured I'd fire out the question here to see if someone can
> give me a hand.
> 
> I'm looking at https://bugzilla.redhat.com/show_bug.cgi?id=485085 . In
> short, I'm seeing an error changing a base channel through the SSM
> when you're changing from the default channel to the default channel
> (I know, it's a weird case).
> 
> In particular BaseSubscribeAction line 166 is what's confusing me.
> 
> If I'm reading this right, we get here if you're changing to the
> default base channel (it will be ID -1, which gets us into the
> condition at 164).
> 
> What I don't get is why we're loading a DistChannelMap for the old
> channel. When my old channel is the default base, that map is coming
> back as non-null. That's causing me to get to line 169, but ends up
> with a null pointer exception at 172.

Starting to remember a bit about this.

So rhnDistChannelMap is used to map to the main RHEL channel for a
given version arch and product. The first dist channel map lookup is
because we don't have access to some of these parameters in this
context for that system (can't remember which). I recall fighting
with this and being unable to query that information any other way.

As per IRC discussion I think the problem is the use of
ChannelManager.RHEL_PRODUCT_NAME in line 169 (the second lookup). That
table stores "Red Hat Linux" as the product name, but RHEL_PRODUCT_NAME
= "rhel" and even has some notes above where it's defined stating how
product name changes from table to table.

Not real sure why this hasn't broken before now...

Still investigating, will hit you up on irc.

Devan

- -- 
  Devan Goodwin <[email protected]>
  Software Engineer     Spacewalk / RHN Satellite
  Halifax, Canada       650.567.9039x79267
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.9 (GNU/Linux)

iEYEARECAAYFAkmTA6oACgkQAyHWaPV9my7/zwCggnbJRTEf4af+md6VV7tAZEql
JRoAninlKQ5sOEp1/uBWIqMPfgsGPbec
=/GzY
-----END PGP SIGNATURE-----

_______________________________________________
Spacewalk-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/spacewalk-devel

Reply via email to