[katello-devel] Spec failures - help?

Tom McKay thomasmckay at redhat.com
Thu Jan 26 20:13:40 UTC 2012 

To further give weight to the inflexibility of /etc, a quick grep of scripts dir will show that it's hard coded there. I believe rake blows up too if it's not there.

----- Original Message -----
> From: "Tom McKay" <thomasmckay at redhat.com>
> To: "Lukas Zapletal" <lzap at redhat.com>
> Cc: katello-devel at redhat.com
> Sent: Thursday, January 26, 2012 11:54:52 AM
> Subject: Re: [katello-devel] Spec failures - help?
> 
> 
> 
> ----- Original Message -----
> > From: "Lukas Zapletal" <lzap at redhat.com>
> > To: katello-devel at redhat.com
> > Sent: Thursday, January 26, 2012 9:12:08 AM
> > Subject: Re: [katello-devel] Spec failures - help?
> > 
> > But we already do this:
> > 
> > - first search for /etc/katello/katello.yml
> > - not found? use KATELLO_LOCATION/config/katello.yml
> 
> This order does not make sense to me. In order to run two versions of
> katello on a system (we're talking R&D here, not customer) the
> /etc/katello/katello.yml must not exist, and the local git version
> must be modified. This seems prone to inadvertently checking in a
> locally modified katello.ym into git.
> 
> Instead, if the git version was called katello.yml.example, then
> changes could be tracked there. With this name change, a locally
> modified version could also live along side it.
> 
> By changing the order to first check locally, you could actually
> install katello on your machine (which would add /etc), but also
> have local versions for testing (w/ local katello.yml).
> 
> If I'm the only one that sees a problem with QA scripts running out
> of /etc instead of local and the potential errors associated with
> this, I'll agree to disagree and live with it. :)
> 
> > 
> > This gives us flexibility of redefining the git defaults. We
> > already
> > discussed this on this list if you want to read more details about
> > it.
> > 
> > For upgrades, I think, we will require explicit upgrading scripts
> > that
> > can do anything. Unfortunately, we do not track in the katello.yml
> > yet.
> > The best approach is to watch for changes in the default git file.
> > 
> > The rule is - if you add a value, always add it there, even
> > commented
> > out.
> > 
> > LZ
> > 
> > On Thu, Jan 26, 2012 at 08:06:53AM -0500, Tom McKay wrote:
> > > On a related note, any changes to katello.yml probably should be
> > > flagged in some way for a customer README or such for when we
> > > support upgrades between versions. These changes are equivalent
> > > to
> > > db schema changes from an install perspective, right?
> > > 
> > > ----- Original Message -----
> > > > From: "Tom McKay" <thomasmckay at redhat.com>
> > > > To: "Ivan Nečas" <inecas at redhat.com>
> > > > Cc: katello-devel at redhat.com
> > > > Sent: Thursday, January 26, 2012 8:00:58 AM
> > > > Subject: Re: [katello-devel] Spec failures - help?
> > > > 
> > > > It's definitely a bad strategy for our local installs to be
> > > > looking
> > > > for anything in /etc, especially when running spec tests. This
> > > > can
> > > > lead, as we are seeing, to false results. (One could argue that
> > > > the
> > > > failures are true failures and that the missing AppConfig.host
> > > > should be handled more gracefully.) My suggestion is to rename
> > > > config/katello.yml to config/katello.yml.example, then have the
> > > > code
> > > > look for it first in local config/katello.yml, then in
> > > > /etc/katello/katello.yml. Any disagreements with that?
> > > > 
> > > > 
> > > > ----- Original Message -----
> > > > > From: "Ivan Nečas" <inecas at redhat.com>
> > > > > To: katello-devel at redhat.com
> > > > > Sent: Thursday, January 26, 2012 4:52:31 AM
> > > > > Subject: Re: [katello-devel] Spec failures - help?
> > > > > 
> > > > > It looks like AppConfig.host being nil in your config when
> > > > > running
> > > > > test
> > > > > environment. Might be problem with your katello.yml host
> > > > > config.
> > > > > What
> > > > > does your katello.yml look like (both in you repo's config
> > > > > and
> > > > > /etc/katello)?
> > > > > 
> > > > > -- Ivan
> > > > > On 01/25/2012 07:26 PM, Jason Rist wrote:
> > > > > > Hi All - Getting a few spec failures and it sounds like
> > > > > > Thomas is
> > > > > > getting the same ones.  Can anyone explain what I'm doing
> > > > > > wrong
> > > > > > or
> > > > > > are
> > > > > > these errors lingering from a change?
> > > > > > Failures:
> > > > > >
> > > > > >    1) Product product repos add repo when there is a repo
> > > > > >    with
> > > > > >    the
> > > > > >    same
> > > > > > name for the product should raise conflict error
> > > > > >       Failure/Error: @p.add_repo(@repo_name,
> > > > > >       "http://test/repo","yum" )
> > > > > >       NoMethodError:
> > > > > >         You have a nil object when you didn't expect it!
> > > > > >         You might have expected an instance of Array.
> > > > > >         The error occurred while evaluating nil.+
> > > > > >       # ./app/models/glue/pulp/repos.rb:547:in
> > > > > >       `yum_gpg_key_url'
> > > > > >       # ./app/models/glue/pulp/repos.rb:59:in
> > > > > >       `create_content'
> > > > > >       # ./app/models/glue/pulp/repos.rb:356:in `add_repo'
> > > > > >       # ./spec/models/product_spec.rb:192
> > > > > >
> > > > > >    2) RepositoriesController other-tests Test gpg create
> > > > > >       Failure/Error: response.should be_success
> > > > > >         expected success? to return true, got false
> > > > > >       #
> > > > > >       ./spec/controllers/repositories_controller_spec.rb:98
> > > > > >
> > > > > > -J
> > > > > 
> > > > > 
> > > > > --
> > > > > Regards,
> > > > > 
> > > > >   Ivan Necas
> > > > >   Red Hat Czech s.r.o. Brno
> > > > > 
> > > > > _______________________________________________
> > > > > katello-devel mailing list
> > > > > katello-devel at redhat.com
> > > > > https://www.redhat.com/mailman/listinfo/katello-devel
> > > > > 
> > > > 
> > > > _______________________________________________
> > > > katello-devel mailing list
> > > > katello-devel at redhat.com
> > > > https://www.redhat.com/mailman/listinfo/katello-devel
> > > > 
> > > 
> > > _______________________________________________
> > > katello-devel mailing list
> > > katello-devel at redhat.com
> > > https://www.redhat.com/mailman/listinfo/katello-devel
> > 
> > --
> > Later,
> > 
> >  Lukas Zapletal | E32E400A
> >  RHN Satellite Engineering
> >  Red Hat Czech s.r.o. Brno
> > 
> > _______________________________________________
> > katello-devel mailing list
> > katello-devel at redhat.com
> > https://www.redhat.com/mailman/listinfo/katello-devel
> > 
> 
> _______________________________________________
> katello-devel mailing list
> katello-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/katello-devel
>

More information about the katello-devel mailing list