qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DISPATCH-976) Allow policy for sources and targets to handle multiple wildcards
Date Wed, 13 Jun 2018 02:09:00 GMT

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

ASF GitHub Bot commented on DISPATCH-976:
-----------------------------------------

Github user codecov-io commented on the issue:

    https://github.com/apache/qpid-dispatch/pull/324
  
    # [Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/324?src=pr&el=h1) Report
    > Merging [#324](https://codecov.io/gh/apache/qpid-dispatch/pull/324?src=pr&el=desc)
into [master](https://codecov.io/gh/apache/qpid-dispatch/commit/8c7e87275fb481f25d8deaa4f639f3a4d1c532d9?src=pr&el=desc)
will **decrease** coverage by `0.01%`.
    > The diff coverage is `100%`.
    
    [![Impacted file tree graph](https://codecov.io/gh/apache/qpid-dispatch/pull/324/graphs/tree.svg?token=rk2Cgd27pP&width=650&height=150&src=pr)](https://codecov.io/gh/apache/qpid-dispatch/pull/324?src=pr&el=tree)
    
    ```diff
    @@            Coverage Diff             @@
    ##           master     #324      +/-   ##
    ==========================================
    - Coverage   86.55%   86.54%   -0.02%     
    ==========================================
      Files          69       69              
      Lines       15466    15464       -2     
    ==========================================
    - Hits        13387    13383       -4     
    - Misses       2079     2081       +2
    ```
    
    
    | [Impacted Files](https://codecov.io/gh/apache/qpid-dispatch/pull/324?src=pr&el=tree)
| Coverage Δ | |
    |---|---|---|
    | [src/policy.c](https://codecov.io/gh/apache/qpid-dispatch/pull/324/diff?src=pr&el=tree#diff-c3JjL3BvbGljeS5j)
| `86.75% <100%> (+0.85%)` | :arrow_up: |
    | [src/router\_core/agent\_link.c](https://codecov.io/gh/apache/qpid-dispatch/pull/324/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2xpbmsuYw==)
| `63.21% <0%> (-0.58%)` | :arrow_down: |
    | [src/router\_core/router\_core.c](https://codecov.io/gh/apache/qpid-dispatch/pull/324/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlcl9jb3JlLmM=)
| `98.88% <0%> (-0.38%)` | :arrow_down: |
    | [src/router\_core/transfer.c](https://codecov.io/gh/apache/qpid-dispatch/pull/324/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3RyYW5zZmVyLmM=)
| `90.26% <0%> (-0.32%)` | :arrow_down: |
    | [src/router\_core/connections.c](https://codecov.io/gh/apache/qpid-dispatch/pull/324/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2Nvbm5lY3Rpb25zLmM=)
| `95.01% <0%> (-0.24%)` | :arrow_down: |
    | [src/router\_node.c](https://codecov.io/gh/apache/qpid-dispatch/pull/324/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9ub2RlLmM=)
| `94.57% <0%> (-0.15%)` | :arrow_down: |
    | [src/server.c](https://codecov.io/gh/apache/qpid-dispatch/pull/324/diff?src=pr&el=tree#diff-c3JjL3NlcnZlci5j)
| `87.73% <0%> (-0.12%)` | :arrow_down: |
    | [src/iterator.c](https://codecov.io/gh/apache/qpid-dispatch/pull/324/diff?src=pr&el=tree#diff-c3JjL2l0ZXJhdG9yLmM=)
| `86.8% <0%> (+0.19%)` | :arrow_up: |
    
    ------
    
    [Continue to review full report at Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/324?src=pr&el=continue).
    > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
    > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
    > Powered by [Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/324?src=pr&el=footer).
Last update [8c7e872...1892f5d](https://codecov.io/gh/apache/qpid-dispatch/pull/324?src=pr&el=lastupdated).
Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).



> Allow policy for sources and targets to handle multiple wildcards
> -----------------------------------------------------------------
>
>                 Key: DISPATCH-976
>                 URL: https://issues.apache.org/jira/browse/DISPATCH-976
>             Project: Qpid Dispatch
>          Issue Type: Bug
>          Components: Policy Engine
>    Affects Versions: 1.0.1
>            Reporter: Chuck Rolke
>            Assignee: Chuck Rolke
>            Priority: Major
>             Fix For: 1.2.0
>
>
> As per the [upstream policy documentation|https://github.com/apache/qpid-dispatch/blob/1.0.0/doc/book/policy.adoc]
in the secion on [AMQP Source and Target Wildcard and Name Substitution|https://github.com/apache/qpid-dispatch/blob/1.0.0/doc/book/policy.adoc#amqp-source-and-target-wildcard-and-name-substitution]
utilization of the asterisk wildcard is constrained as per the following quote:
> {quote}The asterisk is recognized only if it is the last character in the link name.
> {quote}
> It is desirable that the configuration could handle a situation where there are multiple
asterisks in use such as:
>  
> |{color:#000000}"tstGroup": {{color}|
> |{color:#000000} "users": "tst@amq",{color}|
> |{color:#000000} "sources": "abc.*,my.*.b2b.tst.*",{color}|
> |{color:#000000} "targets": "abc.*,my.*.b2b.tst.*",{color}|
> |{color:#000000} "remoteHosts": "*"{color}|
> |{color:#000000}} {color}|
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


Mime
View raw message