Uploaded image for project: 'Hawkular'
  1. Hawkular
  2. HAWKULAR-1090

services distro doesn't set necessary properties anymore

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Major Major
    • 1.0.0.Alpha15
    • None
    • None
    • None

      From my mailing list post:

      ===

      For the record, I just checked out hawkular-services tag 0.0.2.Final, built the dist, and I see this in standalone.xml now:

      <system-properties>
      <property name="hawkular.backend" value="${hawkular.backend:embedded_cassandra}"/>
      <property name="hawkular.agent.enabled" value="${hawkular.agent.enabled:false}"/>
      </system-properties>

      which means all of this will work.

      But now, in master, I don't get any <system-properties> section at all.

      ----- Original Message -----
      > This recently changed and doesn't work like it did before. I can't figure out
      > what's wrong.
      >
      > The -Pdev build isn't working correctly (and neither is the non-dev build).
      >
      > When you build without -Pdev, the agent should be disabled by default. But it
      > is not, and you will get agent errors on startup.
      >
      > When you build with -Pdev, the agent should be enabled and with the
      > hawkualr.rest.user/password properties set inside standalone.xml. But they
      > are not, and you again get agent startup errors.
      >
      > This used to work a few days ago.
      >
      > The .xsl that used to tweak the standalone.xml doesn't seem to be doing what
      > it was supposed to do. Look here:
      >
      > https://github.com/hawkular/hawkular-services/blob/master/feature-pack/src/main/xsl/configuration/standalone/hawkular-services-template.xsl#L30-L51
      >
      > See where it sets the system properties to control the agent enabled flag and
      > the credentials? This isn't doing anyting that I can see because the distro
      > doesn't have any system proeprties in standalone.xml, but this used to work.
      > We'd get those <system-properties> elements in standalone.xml.
      >
      > I can't figure out what changed. Hoping someone knows the deal and can put
      > this back.

            jpkroehling@redhat.com Juraci Paixão Kröhling (Inactive)
            jmazzitelli John Mazzitelli
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: