Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docker compose int-registry-batch-loader failure #41

Closed
gxtchen opened this issue Apr 24, 2022 · 7 comments
Closed

docker compose int-registry-batch-loader failure #41

gxtchen opened this issue Apr 24, 2022 · 7 comments
Assignees
Labels
B12.1 bug Something isn't working i&t.issue s.medium Medium level severity

Comments

@gxtchen
Copy link

gxtchen commented Apr 24, 2022

🐛 Describe the bug identified during I&T

Trying to load Registry API with data following README from https://github.com/NASA-PDS/registry/blob/main/docker/README.md, docker-reg-api-integration-test-with-wait-1 exited with code 1 and no data was loaded

📜 To Reproduce

Steps to reproduce the behavior:

  1. git clone https://github.com/NASA-PDS/registry.git
  2. cd docker/certs
  3. ./generate-certs.sh
  4. cd ..
  5. docker compose --profile=pds-core-registry up -d
  6. docker compose --profile=int-registry-batch-loader up

registry-docker.log

🕵️ Expected behavior

docker-reg-api-integration-1 exited with code 0

⚙️ Engineering Details

@gxtchen gxtchen added bug Something isn't working needs:triage labels Apr 24, 2022
@jordanpadams jordanpadams changed the title Docker for Registry docker compose int-registry-batch-loader failure Apr 25, 2022
@jordanpadams
Copy link
Member

@gxtchen @viviant100 when creating issues you find during I&T, can we try to use the I&T Issue template? https://github.com/NASA-PDS/registry-api/issues/new/choose
Screen Shot 2022-04-25 at 4 22 59 PM

@jordanpadams jordanpadams transferred this issue from NASA-PDS/registry-api Apr 25, 2022
@ramesh-maddegoda
Copy link
Contributor

ramesh-maddegoda commented Apr 26, 2022

@gxtchen , thanks for reporting this!

I also executed the same steps you mentioned and noticed docker-reg-api-integration-test-with-wait-1 exited with code 1. However, that does not mean the data was not loaded. I think we should update the README file to check the following status to validate the successful test data load.


docker-registry-loader-test-init-1 exited with code 0

In the log file you shared, I noticed the the following message.


docker-registry-loader-test-init-1           | [SUMMARY] Elasticsearch URL: http://elasticsearch:9200, index: registry
docker-registry-loader-test-init-1           | [ERROR] Connection is closed

Was Elasticsearch/OpesnSearch URL is specified as http://elasticsearch:9200 (with HTTP) instead of https://elasticsearch:9200 (with HTTPS) in any of the configuration files?

@ramesh-maddegoda
Copy link
Contributor

@gxtchen , also can you please delete the nasapds/registry-loader image in your local docker environment and allow the docker compose to pull the latest docker image of nasapds/registry-loader ?

@tloubrieu-jpl
Copy link
Member

@gxtchen there are currently issues in the way the CICD generates the latest docker images. So I cannot see clearly what is causing the error that you are seeing in the automated integration tests.

I will try to generate a version 1.0.0 of the registry by tomorrow so that you can start the registry using the docker compose framework.

@tloubrieu-jpl
Copy link
Member

Hi @gxtchen @viviant100 this issue is solved in the release 1.0.1 of the registry. The documentation is also published online for the docker compose deployment see https://nasa-pds.github.io/registry/install/docker-compose.html

I hope this helps. Let us know. Thanks

@viviant100
Copy link

@tloubrieu-jpl I&T is done with B12.1 registry I&T. Are you asking for another round of I&T testing for this particular release (1.0.1)?

@tloubrieu-jpl
Copy link
Member

@viviant100 no I don't think we need to do one more round of testing. Sorry that was not ready for @gxtchen to work with the docker compose deployment but i am happy he was not blocked because of that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B12.1 bug Something isn't working i&t.issue s.medium Medium level severity
Projects
None yet
Development

No branches or pull requests

5 participants