You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/02/01 02:27:31 UTC
[GitHub] [pulsar] tuteng opened a new pull request #6184: [Github
actions]Fixed function state and cli integration test
tuteng opened a new pull request #6184: [Github actions]Fixed function state and cli integration test
URL: https://github.com/apache/pulsar/pull/6184
### Motivation
There are a large number of 500 errors in current test integration tests cli and function state. Fix this problem.
### Modifications
* Split integration tests and delete useless images
### Verifying this change
https://github.com/AmateurEvents/pulsar/pull/9 Cli and State tests pass
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [pulsar] sijie merged pull request #6184: [Github actions]Improve
integration test
Posted by GitBox <gi...@apache.org>.
sijie merged pull request #6184: [Github actions]Improve integration test
URL: https://github.com/apache/pulsar/pull/6184
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [pulsar] tuteng edited a comment on issue #6184: [Github
actions]Improve integration test
Posted by GitBox <gi...@apache.org>.
tuteng edited a comment on issue #6184: [Github actions]Improve integration test
URL: https://github.com/apache/pulsar/pull/6184#issuecomment-581020469
You are right, But some of these containers are interdependent. We do not know the order of dependency, so if we delete them directly, we will throw some exceptions, which will cause the test to exit directly. @aahmed-se
```
docker images
REPOSITORY TAG IMAGE ID CREATED SIZE
apachepulsar/pulsar-test-latest-version 2.6.0-SNAPSHOT c599fe91d80e 2 seconds ago 2.82GB
apachepulsar/pulsar-test-latest-version latest c599fe91d80e 2 seconds ago 2.82GB
apachepulsar/pulsar-all latest 77d849fb13b7 3 days ago 2.76GB
node 10 ea119cebc1c3 3 weeks ago 908MB
node 12 6b5991bf650f 3 weeks ago 913MB
buildpack-deps stretch 2adbcf3b6aff 3 weeks ago 835MB
debian 9 f6c68e2ad82a 3 weeks ago 101MB
debian 8 5d7d9c6338e8 3 weeks ago 129MB
node 10-alpine d32bf7fc7d5f 3 weeks ago 80.5MB
node 12-alpine 1cbcaddb8074 3 weeks ago 85.2MB
alpine 3.10 965ea09ff2eb 3 months ago 5.55MB
jekyll/builder latest 4605d94a6b36 3 months ago 564MB
alpine 3.9 055936d39205 8 months ago 5.53MB
alpine 3.7 6d1ef012b567 10 months ago 4.21MB
alpine 3.8 dac705114996 10 months ago 4.41MB
mcr.microsoft.com/azure-pipelines/node8-typescript latest 9a948d360778
```
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [pulsar] tuteng commented on issue #6184: [Github actions]Improve
integration test
Posted by GitBox <gi...@apache.org>.
tuteng commented on issue #6184: [Github actions]Improve integration test
URL: https://github.com/apache/pulsar/pull/6184#issuecomment-581020469
You are right, But some of these containers are interdependent. We do not know the order of dependency, so if we delete them directly, we will throw some exceptions, which will cause the test to exit directly.
```
docker images
REPOSITORY TAG IMAGE ID CREATED SIZE
apachepulsar/pulsar-test-latest-version 2.6.0-SNAPSHOT c599fe91d80e 2 seconds ago 2.82GB
apachepulsar/pulsar-test-latest-version latest c599fe91d80e 2 seconds ago 2.82GB
apachepulsar/pulsar-all latest 77d849fb13b7 3 days ago 2.76GB
node 10 ea119cebc1c3 3 weeks ago 908MB
node 12 6b5991bf650f 3 weeks ago 913MB
buildpack-deps stretch 2adbcf3b6aff 3 weeks ago 835MB
debian 9 f6c68e2ad82a 3 weeks ago 101MB
debian 8 5d7d9c6338e8 3 weeks ago 129MB
node 10-alpine d32bf7fc7d5f 3 weeks ago 80.5MB
node 12-alpine 1cbcaddb8074 3 weeks ago 85.2MB
alpine 3.10 965ea09ff2eb 3 months ago 5.55MB
jekyll/builder latest 4605d94a6b36 3 months ago 564MB
alpine 3.9 055936d39205 8 months ago 5.53MB
alpine 3.7 6d1ef012b567 10 months ago 4.21MB
alpine 3.8 dac705114996 10 months ago 4.41MB
mcr.microsoft.com/azure-pipelines/node8-typescript latest 9a948d360778
```
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [pulsar] aahmed-se commented on issue #6184: [Github
actions]Improve integration test
Posted by GitBox <gi...@apache.org>.
aahmed-se commented on issue #6184: [Github actions]Improve integration test
URL: https://github.com/apache/pulsar/pull/6184#issuecomment-580999713
This is not a good approach. Just delete all container images not used for testing with a simple bash script
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services