You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2022/02/05 00:26:08 UTC

[GitHub] [bookkeeper] nicoloboschi opened a new pull request #3028: [tests] remove backward compatibility test against yahoo-version (apache Pulsar 1.21)

nicoloboschi opened a new pull request #3028:
URL: https://github.com/apache/bookkeeper/pull/3028


   ### Motivation
   
   Currently we have an integration test which download a very old version of Apache Pulsar (I suppose with a BK fork but IDK). 
   
   I think it is better to remove this test because:
   * we don't have control over yahoo repository (it downloads the client from a yahoo repo in very unstable manner)
   * there's not much value upgrading from a unofficial 4.3.x BK fork since we have many other upgrade tests
   * it slows down the docker test image creation and I've personally seen this to be flaky on Github CI 
   
   ### Changes
   
   * Removed all the reference to the yahoo repository and I removed the test itself
   


-- 
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: issues-unsubscribe@bookkeeper.apache.org

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



[GitHub] [bookkeeper] nicoloboschi commented on pull request #3028: [tests] remove backward compatibility test against yahoo-version (apache Pulsar 1.21)

Posted by GitBox <gi...@apache.org>.
nicoloboschi commented on pull request #3028:
URL: https://github.com/apache/bookkeeper/pull/3028#issuecomment-1031558910


   rerun failure checks


-- 
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: issues-unsubscribe@bookkeeper.apache.org

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



[GitHub] [bookkeeper] nicoloboschi commented on pull request #3028: [tests] remove backward compatibility test against yahoo-version (apache Pulsar 1.21)

Posted by GitBox <gi...@apache.org>.
nicoloboschi commented on pull request #3028:
URL: https://github.com/apache/bookkeeper/pull/3028#issuecomment-1032466147






-- 
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: issues-unsubscribe@bookkeeper.apache.org

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



[GitHub] [bookkeeper] nicoloboschi commented on pull request #3028: [tests] remove backward compatibility test against yahoo-version (apache Pulsar 1.21)

Posted by GitBox <gi...@apache.org>.
nicoloboschi commented on pull request #3028:
URL: https://github.com/apache/bookkeeper/pull/3028#issuecomment-1031938874


   rerun failure checks


-- 
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: issues-unsubscribe@bookkeeper.apache.org

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



[GitHub] [bookkeeper] eolivelli merged pull request #3028: [tests] remove backward compatibility test against yahoo-version (apache Pulsar 1.21)

Posted by GitBox <gi...@apache.org>.
eolivelli merged pull request #3028:
URL: https://github.com/apache/bookkeeper/pull/3028


   


-- 
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: issues-unsubscribe@bookkeeper.apache.org

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