[jira] [Created] (SLING-9505) Failure to retry distributing content

2020-06-05 Thread Timothee Maret (Jira)
Timothee Maret created SLING-9505:
-

 Summary: Failure to retry distributing content
 Key: SLING-9505
 URL: https://issues.apache.org/jira/browse/SLING-9505
 Project: Sling
  Issue Type: Bug
  Components: Content Distribution
Reporter: Timothee Maret
Assignee: Timothee Maret
 Fix For: Content Distribution Core 0.4.4


{code}
05.06.2020 20:23:33.066 *ERROR* [127.0.0.1 [1591388613038] POST 
/libs/sling/distribution/services/agents/publish-test HTTP/1.1] 
org.apache.sling.distribution.agent.impl.SimpleDistributionAgent 
[agent][publish-test] an error happened during package import
org.apache.sling.distribution.common.DistributionException: 
org.apache.http.client.ClientProtocolException
at 
org.apache.sling.distribution.transport.impl.SimpleHttpDistributionTransport.deliverPackage(SimpleHttpDistributionTransport.java:164)
 [org.apache.sling.distribution.core:0.4.3.T202004281829-70214c0]
at 
org.apache.sling.distribution.packaging.impl.importer.RemoteDistributionPackageImporter.importPackage(RemoteDistributionPackageImporter.java:69)
 [org.apache.sling.distribution.core:0.4.3.T202004281829-70214c0]
at 
org.apache.sling.distribution.agent.impl.ImportingDistributionPackageProcessor.process(ImportingDistributionPackageProcessor.java:78)
 [org.apache.sling.distribution.core:0.4.3.T202004281829-70214c0]
at 
org.apache.sling.distribution.packaging.impl.exporter.LocalDistributionPackageExporter.exportPackages(LocalDistributionPackageExporter.java:47)
 [org.apache.sling.distribution.core:0.4.3.T202004281829-70214c0]
at 
org.apache.sling.distribution.agent.impl.SimpleDistributionAgent.exportPackages(SimpleDistributionAgent.java:218)
 [org.apache.sling.distribution.core:0.4.3.T202004281829-70214c0]
at 
org.apache.sling.distribution.agent.impl.SimpleDistributionAgent.execute(SimpleDistributionAgent.java:182)
 [org.apache.sling.distribution.core:0.4.3.T202004281829-70214c0]
at 
org.apache.sling.distribution.servlet.DistributionAgentServlet.doPost(DistributionAgentServlet.java:62)
 [org.apache.sling.distribution.core:0.4.3.T202004281829-70214c0]
at 
org.apache.sling.api.servlets.SlingAllMethodsServlet.mayService(SlingAllMethodsServlet.java:146)
 [org.apache.sling.api:2.22.0]
at 
org.apache.sling.api.servlets.SlingSafeMethodsServlet.service(SlingSafeMethodsServlet.java:342)
 [org.apache.sling.api:2.22.0]
at 
org.apache.sling.api.servlets.SlingSafeMethodsServlet.service(SlingSafeMethodsServlet.java:374)
 [org.apache.sling.api:2.22.0]
at 
org.apache.sling.engine.impl.request.RequestData.service(RequestData.java:552) 
[org.apache.sling.engine:2.7.2]
at 
org.apache.sling.engine.impl.filter.SlingComponentFilterChain.render(SlingComponentFilterChain.java:44)
 [org.apache.sling.engine:2.7.2]
at 
org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:82)
 [org.apache.sling.engine:2.7.2]
at 
com.day.cq.wcm.core.impl.WCMDebugFilter.doFilter(WCMDebugFilter.java:138) 
[com.day.cq.wcm.cq-wcm-core:5.13.122]
at 
org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:72)
 [org.apache.sling.engine:2.7.2]
at 
org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:78)
 [org.apache.sling.engine:2.7.2]
at 
com.day.cq.wcm.core.impl.WCMComponentFilter.filterRootInclude(WCMComponentFilter.java:375)
 [com.day.cq.wcm.cq-wcm-core:5.13.122]
at 
com.day.cq.wcm.core.impl.WCMComponentFilter.doFilter(WCMComponentFilter.java:190)
 [com.day.cq.wcm.cq-wcm-core:5.13.122]
at 
org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:72)
 [org.apache.sling.engine:2.7.2]
at 
com.day.cq.wcm.core.impl.page.PageLockFilter.doFilter(PageLockFilter.java:91) 
[com.day.cq.wcm.cq-wcm-core:5.13.122]
at 
org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:72)
 [org.apache.sling.engine:2.7.2]
at 
com.day.cq.personalization.impl.TargetComponentFilter.doFilter(TargetComponentFilter.java:94)
 [com.day.cq.cq-personalization:5.13.14]
at 
org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:72)
 [org.apache.sling.engine:2.7.2]
at 
org.apache.sling.engine.impl.SlingRequestProcessorImpl.processComponent(SlingRequestProcessorImpl.java:283)
 [org.apache.sling.engine:2.7.2]
at 
org.apache.sling.engine.impl.filter.RequestSlingFilterChain.render(RequestSlingFilterChain.java:49)
 [org.apache.sling.engine:2.7.2]
at 
org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:76)
 [org.apache.sling.engine:2.7.2]
at 
org.apache.sl

[tooling] Launching feature model applications in testing

2020-06-05 Thread Robert Munteanu
Hi,

For SLING-9499 [1] I am looking into launching a feature model
application as part of a Maven build, for integration testing.

I have looked into what the current Maven tooling offers and found:

- slingfeature-maven-plugin:launch-features, that launches a single
application, in-process, blocking, based on one or more feature files
- sling-kickstart-maven-plugin:start, that launches one or more
kickstart jars, out-of-process, non-blocking

What I am looking for is something in the middle: launch a single
application, based on feature files, out-of-process, non-blocking. I
guess adding multiple applications won't be that hard.

I think that we can enhance the Maven tooling to better support
starting and stopping 'pure' feature model applications. This will be
useful for people developing on top of the feature model but not with
Sling. Then we can rebase the kickstart tooling to take advantage of
the new enhancements.

Thoughts?

Thanks,
Robert

[1]: https://issues.apache.org/jira/browse/SLING-9499



[jira] [Commented] (SLING-4443) Whitespace removal for Sightly HTML output

2020-06-05 Thread Jira


[ 
https://issues.apache.org/jira/browse/SLING-4443?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17127088#comment-17127088
 ] 

Jörg Hoh commented on SLING-4443:
-

This is still an issue; if have here a HTML page which is 30 MB (!!!) with > 25 
MB of just whitespace. Of course mod_deflate makes that bearable, but it would 
definitely speed up a lot of processes if the whitespace would be removed.

> Whitespace removal for Sightly HTML output
> --
>
> Key: SLING-4443
> URL: https://issues.apache.org/jira/browse/SLING-4443
> Project: Sling
>  Issue Type: New Feature
>  Components: Scripting
>Affects Versions: Scripting Sightly Engine 1.0.0
>Reporter: Dan Chapman
>Priority: Minor
>
> Can there be a way to remove whitespace from Sightly components HTML output, 
> that is  created when elements are hidden e.g.  Previously in JSPs we could use - trimDirectiveWhitespaces="true" 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Assigned] (SLING-9499) Launchpad-Testing: branch which runs ITs against oak_tar feature model aggregate

2020-06-05 Thread Robert Munteanu (Jira)


 [ 
https://issues.apache.org/jira/browse/SLING-9499?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Munteanu reassigned SLING-9499:
--

Assignee: Robert Munteanu

> Launchpad-Testing: branch which runs ITs against oak_tar feature model 
> aggregate
> 
>
> Key: SLING-9499
> URL: https://issues.apache.org/jira/browse/SLING-9499
> Project: Sling
>  Issue Type: Sub-task
>  Components: Starter, Testing
>Reporter: Robert Munteanu
>Assignee: Robert Munteanu
>Priority: Major
> Fix For: Launchpad Testing 12
>
>
> Create branch running ITs against the oak_tar aggregate created in the 
> previous step



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [sling-org-apache-sling-starter] rombert merged pull request #10: SLING-9498 - Create feature model aggregates on the fly from the Sling Starter build

2020-06-05 Thread GitBox


rombert merged pull request #10:
URL: https://github.com/apache/sling-org-apache-sling-starter/pull/10


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[jira] [Resolved] (SLING-9498) Create feature model aggregates on the fly from the Sling Starter build

2020-06-05 Thread Robert Munteanu (Jira)


 [ 
https://issues.apache.org/jira/browse/SLING-9498?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Munteanu resolved SLING-9498.

Resolution: Fixed

Fixed with https://github.com/apache/sling-org-apache-sling-starter/pull/10

> Create feature model aggregates on the fly from the Sling Starter build
> ---
>
> Key: SLING-9498
> URL: https://issues.apache.org/jira/browse/SLING-9498
> Project: Sling
>  Issue Type: Sub-task
>  Components: Starter
>Reporter: Robert Munteanu
>Assignee: Robert Munteanu
>Priority: Major
> Fix For: Starter 12
>
>  Time Spent: 1h
>  Remaining Estimate: 0h
>
> Create feature model aggregates on the fly from the build - oak_tar and 
> oak_mongo. This should use the feature-converter-maven-plugin to create the 
> feature files.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [sling-org-apache-sling-starter] sonarcloud[bot] commented on pull request #10: SLING-9498 - Create feature model aggregates on the fly from the Sling Starter build

2020-06-05 Thread GitBox


sonarcloud[bot] commented on pull request #10:
URL: 
https://github.com/apache/sling-org-apache-sling-starter/pull/10#issuecomment-639777887


   Kudos, SonarCloud Quality Gate passed!
   
   [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-starter&pullRequest=10&resolved=false&types=BUG)
 [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-starter&pullRequest=10&resolved=false&types=BUG)
 [0 
Bugs](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-starter&pullRequest=10&resolved=false&types=BUG)
  
   [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-starter&pullRequest=10&resolved=false&types=VULNERABILITY)
 [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-starter&pullRequest=10&resolved=false&types=VULNERABILITY)
 [0 
Vulnerabilities](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-starter&pullRequest=10&resolved=false&types=VULNERABILITY)
 (and [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-starter&pullRequest=10&resolved=false&types=SECURITY_HOTSPOT)
 [0 Security 
Hotspots](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-starter&pullRequest=10&resolved=false&types=SECURITY_HOTSPOT)
 to review)  
   [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-starter&pullRequest=10&resolved=false&types=CODE_SMELL)
 [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-starter&pullRequest=10&resolved=false&types=CODE_SMELL)
 [0 Code 
Smells](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-starter&pullRequest=10&resolved=false&types=CODE_SMELL)
   
   [](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-starter&pullRequest=10)
 No Coverage information  
   [](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-starter&pullRequest=10&metric=new_duplicated_lines_density&view=list)
 [0.0% 
Duplication](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-starter&pullRequest=10&metric=new_duplicated_lines_density&view=list)
   
The version of Java (1.8.0_241) you 
have used to run this analysis is deprecated and we will stop accepting it from 
October 2020. Please update to at least Java 11.
   Read more [here](https://sonarcloud.io/documentation/upcoming/)
   
   
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [sling-org-apache-sling-starter] rombert opened a new pull request #10: SLING-9498 - Create feature model aggregates on the fly from the Sling Starter build

2020-06-05 Thread GitBox


rombert opened a new pull request #10:
URL: https://github.com/apache/sling-org-apache-sling-starter/pull/10


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[jira] [Commented] (SLING-9489) Make it possible to compare parts of feature models

2020-06-05 Thread Carsten Ziegeler (Jira)


[ 
https://issues.apache.org/jira/browse/SLING-9489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17126866#comment-17126866
 ] 

Carsten Ziegeler commented on SLING-9489:
-

As all of that is metadata, you could add a (boolean) configuration flag for 
comparing metadata
But I would think that by default this should be off

> Make it possible to compare parts of feature models
> ---
>
> Key: SLING-9489
> URL: https://issues.apache.org/jira/browse/SLING-9489
> Project: Sling
>  Issue Type: New Feature
>  Components: Feature Model, Maven Plugins and Archetypes
>Affects Versions: slingfeature-maven-plugin 1.3.4
>Reporter: A. J. David Bosschaert
>Assignee: A. J. David Bosschaert
>Priority: Major
> Fix For: slingfeature-maven-plugin 1.3.6
>
>
> We need a mojo that allows the comparison of parts of a feature model. If 
> these parts are not equal the mojo should fail the build.
> Right now comparing extensions would be a good start. But comparing 
> configurations or bundles would also be good.
> Key is that not everything needs to be compared. The configuration in the 
> pom.xml should specify what is being compared.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (SLING-9489) Make it possible to compare parts of feature models

2020-06-05 Thread A. J. David Bosschaert (Jira)


[ 
https://issues.apache.org/jira/browse/SLING-9489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17126859#comment-17126859
 ] 

A. J. David Bosschaert commented on SLING-9489:
---

What about Artifact Metadata, aliases and start order? Should these match in 
the comparison?

> Make it possible to compare parts of feature models
> ---
>
> Key: SLING-9489
> URL: https://issues.apache.org/jira/browse/SLING-9489
> Project: Sling
>  Issue Type: New Feature
>  Components: Feature Model, Maven Plugins and Archetypes
>Affects Versions: slingfeature-maven-plugin 1.3.4
>Reporter: A. J. David Bosschaert
>Assignee: A. J. David Bosschaert
>Priority: Major
> Fix For: slingfeature-maven-plugin 1.3.6
>
>
> We need a mojo that allows the comparison of parts of a feature model. If 
> these parts are not equal the mojo should fail the build.
> Right now comparing extensions would be a good start. But comparing 
> configurations or bundles would also be good.
> Key is that not everything needs to be compared. The configuration in the 
> pom.xml should specify what is being compared.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Assigned] (SLING-9495) Distributed Event is sending package type as distribution type

2020-06-05 Thread Dirk Rudolph (Jira)


 [ 
https://issues.apache.org/jira/browse/SLING-9495?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Dirk Rudolph reassigned SLING-9495:
---

Assignee: Dirk Rudolph

> Distributed Event is sending package type as distribution type
> --
>
> Key: SLING-9495
> URL: https://issues.apache.org/jira/browse/SLING-9495
> Project: Sling
>  Issue Type: Bug
>  Components: Content Distribution
>Affects Versions: Content Distribution Journal Core 0.1.10
>Reporter: Dirk Rudolph
>Assignee: Dirk Rudolph
>Priority: Major
>
> When using journal based content distribution the distributed event contains 
> the package type instead of the distribution request type as distribution 
> type.
> See the following series of events on the DistributionPublisher (author)
> |distribution.type|ADD|
> |distribution.component.kind|agent|
> |event.topics|*org/apache/sling/distribution/agent/package/created*|
> |distribution.package.id|dstrpck-159806534-03e0b77c-62fb-4781-8ec0-462a8a2e7e40|
> |distribution.paths|/content/wknd/ch/de|
> |distribution.component.name|journal|
> |distribution.type|ADD|
> |distribution.component.kind|agent|
> |event.topics|*org/apache/sling/distribution/agent/package/queued*|
> |distribution.package.id|dstrpck-159806534-03e0b77c-62fb-4781-8ec0-462a8a2e7e40
> |distribution.paths|/content/wknd/ch/de|
> |distribution.component.name|journal|
> |distribution.type|{color:red}default{color}|
> |distribution.component.kind|agent|
> |event.topics|*org/apache/sling/distribution/agent/package/distributed*|
> |distribution.package.id|dstrpck-159806534-03e0b77c-62fb-4781-8ec0-462a8a2e7e40|
> |distribution.paths|/content/wknd/ch/de|
> |distribution.component.name|journal|



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Created] (SLING-9504) Switch from protobuf to json

2020-06-05 Thread Christian Schneider (Jira)
Christian Schneider created SLING-9504:
--

 Summary: Switch from protobuf to json
 Key: SLING-9504
 URL: https://issues.apache.org/jira/browse/SLING-9504
 Project: Sling
  Issue Type: Improvement
  Components: Content Distribution
Affects Versions: Content Distribution Journal Messages 0.1.2
Reporter: Christian Schneider
Assignee: Christian Schneider
 Fix For: Content Distribution Journal Messages 0.1.8






--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Resolved] (SLING-9501) Packages provided by a non-api-region enabled feature should be on par with the global region

2020-06-05 Thread A. J. David Bosschaert (Jira)


 [ 
https://issues.apache.org/jira/browse/SLING-9501?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

A. J. David Bosschaert resolved SLING-9501.
---
Resolution: Fixed

> Packages provided by a non-api-region enabled feature should be on par with 
> the global region
> -
>
> Key: SLING-9501
> URL: https://issues.apache.org/jira/browse/SLING-9501
> Project: Sling
>  Issue Type: Bug
>  Components: Feature Model
>Affects Versions: Feature Model API Regions Runtime Fragment 1.1.2
>Reporter: A. J. David Bosschaert
>Assignee: A. J. David Bosschaert
>Priority: Major
> Fix For: Feature Model API Regions Runtime Fragment 1.1.4
>
>
> The API Regions Runtime prefers packages coming from bundles that are in a 
> specific region to packages coming from the global region, if this specific 
> region is accessible to the requirer. This also applies to packages provided 
> from a bundle where this package is feature-local.
> However, when a feature is not opting into the API Regions mechanism this 
> should not be applied. When a feature is not taking part in the API Regions 
> it should be considered to provide its packages in the global region and no 
> preference should be applied.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [sling-org-apache-sling-feature-apiregions] bosschaert merged pull request #11: SLING-9501 Packages provided by a non-api-region enabled feature should be on par with the global region

2020-06-05 Thread GitBox


bosschaert merged pull request #11:
URL: https://github.com/apache/sling-org-apache-sling-feature-apiregions/pull/11


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[Jenkins] Sling » sling-org-apache-sling-launchpad-testing » master #830 is FIXED

2020-06-05 Thread Apache Jenkins Server
Please see 
https://builds.apache.org/job/Sling/job/sling-org-apache-sling-launchpad-testing/job/master/830/
 for details.

No further emails will be sent until the status of the build is changed.

[GitHub] [sling-org-apache-sling-feature-apiregions] sonarcloud[bot] removed a comment on pull request #11: SLING-9501 Packages provided by a non-api-region enabled feature should be on par with the g

2020-06-05 Thread GitBox


sonarcloud[bot] removed a comment on pull request #11:
URL: 
https://github.com/apache/sling-org-apache-sling-feature-apiregions/pull/11#issuecomment-639399295


   Kudos, SonarCloud Quality Gate passed!
   
   [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=BUG)
 [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=BUG)
 [0 
Bugs](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=BUG)
  
   [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=VULNERABILITY)
 [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=VULNERABILITY)
 [0 
Vulnerabilities](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=VULNERABILITY)
 (and [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=SECURITY_HOTSPOT)
 [0 Security 
Hotspots](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=SECURITY_HOTSPOT)
 to review)  
   [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=CODE_SMELL)
 [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=CODE_SMELL)
 [2 Code 
Smells](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=CODE_SMELL)
   
   [](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&metric=new_coverage&view=list)
 [86.7% 
Coverage](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&metric=new_coverage&view=list)
  
   [](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&metric=new_duplicated_lines_density&view=list)
 [0.0% 
Duplication](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&metric=new_duplicated_lines_density&view=list)
   
The version of Java (1.8.0_241) you 
have used to run this analysis is deprecated and we will stop accepting it from 
October 2020. Please update to at least Java 11.
   Read more [here](https://sonarcloud.io/documentation/upcoming/)
   
   
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [sling-org-apache-sling-feature-apiregions] sonarcloud[bot] commented on pull request #11: SLING-9501 Packages provided by a non-api-region enabled feature should be on par with the global re

2020-06-05 Thread GitBox


sonarcloud[bot] commented on pull request #11:
URL: 
https://github.com/apache/sling-org-apache-sling-feature-apiregions/pull/11#issuecomment-639475246


   Kudos, SonarCloud Quality Gate passed!
   
   [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=BUG)
 [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=BUG)
 [0 
Bugs](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=BUG)
  
   [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=VULNERABILITY)
 [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=VULNERABILITY)
 [0 
Vulnerabilities](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=VULNERABILITY)
 (and [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=SECURITY_HOTSPOT)
 [0 Security 
Hotspots](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=SECURITY_HOTSPOT)
 to review)  
   [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=CODE_SMELL)
 [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=CODE_SMELL)
 [0 Code 
Smells](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=CODE_SMELL)
   
   [](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&metric=new_coverage&view=list)
 [86.4% 
Coverage](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&metric=new_coverage&view=list)
  
   [](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&metric=new_duplicated_lines_density&view=list)
 [0.0% 
Duplication](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&metric=new_duplicated_lines_density&view=list)
   
The version of Java (1.8.0_241) you 
have used to run this analysis is deprecated and we will stop accepting it from 
October 2020. Please update to at least Java 11.
   Read more [here](https://sonarcloud.io/documentation/upcoming/)
   
   
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [sling-org-apache-sling-feature-apiregions] sonarcloud[bot] commented on pull request #11: SLING-9501 Packages provided by a non-api-region enabled feature should be on par with the global re

2020-06-05 Thread GitBox


sonarcloud[bot] commented on pull request #11:
URL: 
https://github.com/apache/sling-org-apache-sling-feature-apiregions/pull/11#issuecomment-639399295


   Kudos, SonarCloud Quality Gate passed!
   
   [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=BUG)
 [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=BUG)
 [0 
Bugs](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=BUG)
  
   [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=VULNERABILITY)
 [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=VULNERABILITY)
 [0 
Vulnerabilities](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=VULNERABILITY)
 (and [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=SECURITY_HOTSPOT)
 [0 Security 
Hotspots](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=SECURITY_HOTSPOT)
 to review)  
   [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=CODE_SMELL)
 [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=CODE_SMELL)
 [2 Code 
Smells](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&resolved=false&types=CODE_SMELL)
   
   [](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&metric=new_coverage&view=list)
 [86.7% 
Coverage](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&metric=new_coverage&view=list)
  
   [](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&metric=new_duplicated_lines_density&view=list)
 [0.0% 
Duplication](https://sonarcloud.io/component_measures?id=apache_sling-org-apache-sling-feature-apiregions&pullRequest=11&metric=new_duplicated_lines_density&view=list)
   
The version of Java (1.8.0_241) you 
have used to run this analysis is deprecated and we will stop accepting it from 
October 2020. Please update to at least Java 11.
   Read more [here](https://sonarcloud.io/documentation/upcoming/)
   
   
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [sling-org-apache-sling-feature-apiregions] bosschaert opened a new pull request #11: SLING-9501 Packages provided by a non-api-region enabled feature should be on par with the global region

2020-06-05 Thread GitBox


bosschaert opened a new pull request #11:
URL: https://github.com/apache/sling-org-apache-sling-feature-apiregions/pull/11


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[jira] [Resolved] (SLING-9503) Add control over variable replacement in feature models

2020-06-05 Thread Carsten Ziegeler (Jira)


 [ 
https://issues.apache.org/jira/browse/SLING-9503?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carsten Ziegeler resolved SLING-9503.
-
Resolution: Fixed

The replacement mechanism by default now only replaces project.artifactId, 
project.groupId, project.version and project.osgiVersion

Added three new configuration properties
- enableLegacyVariableReplacement : if set to true, the old mechanism is back 
in place
- enableProjectVariableReplacement : if set to false, the above project 
variables are not replaced
- replacePropertyVariables : comma separated list of properties from the maven 
project that are used in the replacement as well


> Add control over variable replacement in feature models
> ---
>
> Key: SLING-9503
> URL: https://issues.apache.org/jira/browse/SLING-9503
> Project: Sling
>  Issue Type: Improvement
>  Components: Feature Model, Maven Plugins and Archetypes
>Reporter: Carsten Ziegeler
>Assignee: Carsten Ziegeler
>Priority: Major
> Fix For: slingfeature-maven-plugin 1.3.6
>
>
> When a project feature model is read, a variable replacement based on the 
> maven project is performed over the whole feature text file
> While the initial idea was to support "project.*" variables in the "id" of 
> the feature, it ended up as a way too flexible replacement mechanism
> Variables are replaced in an unpredictable way, especially as also system 
> properties are used for replacement - which might result in having a 
> different replacement on different builds.
> We should limit the replacement to the feature id - and only support 
> "project.*" variables there - and to a *configured* set of properties.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Updated] (SLING-9503) Add control over variable replacement in feature models

2020-06-05 Thread Carsten Ziegeler (Jira)


 [ 
https://issues.apache.org/jira/browse/SLING-9503?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carsten Ziegeler updated SLING-9503:

Fix Version/s: (was: Slingstart Maven Plugin 1.9.14)

> Add control over variable replacement in feature models
> ---
>
> Key: SLING-9503
> URL: https://issues.apache.org/jira/browse/SLING-9503
> Project: Sling
>  Issue Type: Improvement
>  Components: Feature Model, Maven Plugins and Archetypes
>Reporter: Carsten Ziegeler
>Assignee: Carsten Ziegeler
>Priority: Major
> Fix For: slingfeature-maven-plugin 1.3.6
>
>
> When a project feature model is read, a variable replacement based on the 
> maven project is performed over the whole feature text file
> While the initial idea was to support "project.*" variables in the "id" of 
> the feature, it ended up as a way too flexible replacement mechanism
> Variables are replaced in an unpredictable way, especially as also system 
> properties are used for replacement - which might result in having a 
> different replacement on different builds.
> We should limit the replacement to the feature id - and only support 
> "project.*" variables there - and to a *configured* set of properties.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Created] (SLING-9503) Add control over variable replacement in feature models

2020-06-05 Thread Carsten Ziegeler (Jira)
Carsten Ziegeler created SLING-9503:
---

 Summary: Add control over variable replacement in feature models
 Key: SLING-9503
 URL: https://issues.apache.org/jira/browse/SLING-9503
 Project: Sling
  Issue Type: Improvement
  Components: Feature Model, Maven Plugins and Archetypes
Reporter: Carsten Ziegeler
Assignee: Carsten Ziegeler
 Fix For: Slingstart Maven Plugin 1.9.14, slingfeature-maven-plugin 
1.3.6


When a project feature model is read, a variable replacement based on the maven 
project is performed over the whole feature text file
While the initial idea was to support "project.*" variables in the "id" of the 
feature, it ended up as a way too flexible replacement mechanism
Variables are replaced in an unpredictable way, especially as also system 
properties are used for replacement - which might result in having a different 
replacement on different builds.

We should limit the replacement to the feature id - and only support 
"project.*" variables there - and to a *configured* set of properties.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)