You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by GitBox <gi...@apache.org> on 2022/10/08 20:44:03 UTC

[GitHub] [james-project] jeantil opened a new pull request, #1239: [JAMES-3805] fixes more flaky pulsar tests

jeantil opened a new pull request, #1239:
URL: https://github.com/apache/james-project/pull/1239

   supercedes https://github.com/apache/james-project/pull/1236
   
   without this patch repeat run of pulsar mailqueue sometimes fails within a dozen runs with a missing `public` tenant. With the patch I couldn't make the run fail on my laptop (stopped a bit over 40 runs)


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org


[GitHub] [james-project] jeantil commented on pull request #1239: [JAMES-3805] fixes more flaky pulsar tests

Posted by GitBox <gi...@apache.org>.
jeantil commented on PR #1239:
URL: https://github.com/apache/james-project/pull/1239#issuecomment-1279770236

   :green_apple: rebase and merge incoming


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org


[GitHub] [james-project] chibenwa commented on pull request #1239: [JAMES-3805] fixes more flaky pulsar tests

Posted by GitBox <gi...@apache.org>.
chibenwa commented on PR #1239:
URL: https://github.com/apache/james-project/pull/1239#issuecomment-1272950964

   > Unfortunately I'm traveling for work this week not sure I can fix it before
   next weekend
   Feel free to take over if you want
   
   Don't worry, I merged the hotfix (`@Tag(Unstable`) so that you have all your time fixing this ;-)


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org


[GitHub] [james-project] jeantil commented on pull request #1239: [JAMES-3805] fixes more flaky pulsar tests

Posted by GitBox <gi...@apache.org>.
jeantil commented on PR #1239:
URL: https://github.com/apache/james-project/pull/1239#issuecomment-1278647527

   Hmmm   I didn't change permissions defaulted by GitHub on the fork but the
   branch  is on my personal fork so isn't that normal? (Also I don't think I
   forked directly from the apache repo but from a fork of the repo or even a
   fork of a fork maybe that's why ? No idea really)
   
   
   Le lun. 10 oct. 2022 à 10:23, Benoit TELLIER ***@***.***> a
   écrit :
   
   > BTW I did not had the right to push on your branch despite it being opened
   > against an ASF repo :-(
   >
   > —
   > Reply to this email directly, view it on GitHub
   > <https://github.com/apache/james-project/pull/1239#issuecomment-1272951779>,
   > or unsubscribe
   > <https://github.com/notifications/unsubscribe-auth/AAAFTQYDW53NTXY5KNCDX3DWCPG7BANCNFSM6AAAAAARAMUCOA>
   > .
   > You are receiving this because you were assigned.Message ID:
   > ***@***.***>
   >
   


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org


[GitHub] [james-project] jeantil merged pull request #1239: [JAMES-3805] fixes more flaky pulsar tests

Posted by GitBox <gi...@apache.org>.
jeantil merged PR #1239:
URL: https://github.com/apache/james-project/pull/1239


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org


[GitHub] [james-project] chibenwa commented on pull request #1239: [JAMES-3805] fixes more flaky pulsar tests

Posted by GitBox <gi...@apache.org>.
chibenwa commented on PR #1239:
URL: https://github.com/apache/james-project/pull/1239#issuecomment-1272951779

   BTW I did not had the right to push on your branch despite it being opened against an ASF repo :-(


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org


[GitHub] [james-project] jeantil commented on pull request #1239: [JAMES-3805] fixes more flaky pulsar tests

Posted by GitBox <gi...@apache.org>.
jeantil commented on PR #1239:
URL: https://github.com/apache/james-project/pull/1239#issuecomment-1272788018

   Hmm sorry about that.
   
   Unfortunately I'm traveling for work this week not sure I can fix it before
   next weekend
   Feel free to take over if you want
   
   
   Le lun. 10 oct. 2022 à 04:26, Rene Cordier ***@***.***> a
   écrit :
   
   > Little issue at the compilation first though, but that should be easily
   > solved ;)
   >
   > 20:56:34,289 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile (default-testCompile) on project scaling-pulsar-smtp: Compilation failure
   > 20:56:34,289 [ERROR] /home/jenkins/workspace/james_ApacheJames_PR-1239/server/apps/scaling-pulsar-smtp/src/test/java/org/apache/james/PulsarExtension.java:[40,25] unreported exception org.apache.pulsar.client.admin.PulsarAdminException; must be caught or declared to be thrown
   >
   > —
   > Reply to this email directly, view it on GitHub
   > <https://github.com/apache/james-project/pull/1239#issuecomment-1272717927>,
   > or unsubscribe
   > <https://github.com/notifications/unsubscribe-auth/AAAFTQYLJVLWIYMQP75BK4DWCN5EVANCNFSM6AAAAAARAMUCOA>
   > .
   > You are receiving this because you were assigned.Message ID:
   > ***@***.***>
   >
   


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org


[GitHub] [james-project] Arsnael commented on pull request #1239: [JAMES-3805] fixes more flaky pulsar tests

Posted by GitBox <gi...@apache.org>.
Arsnael commented on PR #1239:
URL: https://github.com/apache/james-project/pull/1239#issuecomment-1272717927

   Little issue at the compilation first though, but that should be easily solved ;)
   
   ```
   20:56:34,289 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile (default-testCompile) on project scaling-pulsar-smtp: Compilation failure
   20:56:34,289 [ERROR] /home/jenkins/workspace/james_ApacheJames_PR-1239/server/apps/scaling-pulsar-smtp/src/test/java/org/apache/james/PulsarExtension.java:[40,25] unreported exception org.apache.pulsar.client.admin.PulsarAdminException; must be caught or declared to be thrown
   ```


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org