Skip to content
This repository has been archived by the owner on Jan 19, 2018. It is now read-only.

Fix namespace conflicts in answers.conf. Fixes #478 #511

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

rtnpro
Copy link
Contributor

@rtnpro rtnpro commented Jan 15, 2016

Fixes #478

@rtnpro rtnpro added this to the CDK 2 GA milestone Jan 15, 2016
@cdrage
Copy link
Member

cdrage commented Jan 19, 2016

Error on tests. Asks for answers despite being supplied:

2016-01-19 18:52:30,994 - atomicapp.nulecule.container - DEBUG - Copying data from Docker container: /usr/bin/docker cp 51bd07885dc09369e1816f9500ecf18cbc1ce7b4219dd1fd7f3cd562b5b3865b:/application-entity /tmp/nulecule-cb255306-bedd-11e5-a09b-3402868080db
2016-01-19 18:52:31,061 - atomicapp.nulecule.container - DEBUG - Copying nulecule data from /tmp/nulecule-cb255306-bedd-11e5-a09b-3402868080db to build
2016-01-19 18:52:31,064 - atomicapp.nulecule.container - DEBUG - Removing tmp dir: /tmp/nulecule-cb255306-bedd-11e5-a09b-3402868080db
2016-01-19 18:52:31,064 - atomicapp.utils - DEBUG - Recursively removing directory: /tmp/nulecule-cb255306-bedd-11e5-a09b-3402868080db
2016-01-19 18:52:31,066 - atomicapp.nulecule.container - DEBUG - Removing Docker container: /usr/bin/docker rm -f 51bd07885dc09369e1816f9500ecf18cbc1ce7b4219dd1fd7f3cd562b5b3865b
51bd07885dc09369e1816f9500ecf18cbc1ce7b4219dd1fd7f3cd562b5b3865b
2016-01-19 18:52:31,135 - atomicapp.plugin - DEBUG - Loading providers from /opt/atomicapp/atomicapp/providers
2016-01-19 18:52:31,137 - atomicapp.plugin - DEBUG - Loading providers from /opt/atomicapp/atomicapp/providers
2016-01-19 18:52:31,138 - atomicapp.nulecule.base - INFO - Pulling external application for aggregated-mariadb-atomicapp.
2016-01-19 18:52:31,138 - atomicapp.nulecule.base - INFO - Unpacking image: projectatomic/mariadb-centos7-atomicapp to build/external/aggregated-mariadb-atomicapp
2016-01-19 18:52:33,854 - atomicapp.nulecule.container - INFO - Skipping pulling Docker image: projectatomic/mariadb-centos7-atomicapp
2016-01-19 18:52:33,854 - atomicapp.nulecule.container - INFO - Extracting nulecule data from image: projectatomic/mariadb-centos7-atomicapp to build/external/aggregated-mariadb-atomicapp
2016-01-19 18:52:33,854 - atomicapp.nulecule.container - DEBUG - Creating docker container: /usr/bin/docker create --entrypoint /bin/true projectatomic/mariadb-centos7-atomicapp
2016-01-19 18:52:33,964 - atomicapp.nulecule.container - DEBUG - Copying data from Docker container: /usr/bin/docker cp b2251700ecc32b8cb10d01ce29e92034ac3f10a4f225b957c6f0d87e0220d5cf:/application-entity /tmp/nulecule-cceaa7b8-bedd-11e5-a09b-3402868080db
2016-01-19 18:52:34,033 - atomicapp.nulecule.container - DEBUG - Copying nulecule data from /tmp/nulecule-cceaa7b8-bedd-11e5-a09b-3402868080db to build/external/aggregated-mariadb-atomicapp
2016-01-19 18:52:34,035 - atomicapp.nulecule.container - DEBUG - Removing tmp dir: /tmp/nulecule-cceaa7b8-bedd-11e5-a09b-3402868080db
2016-01-19 18:52:34,035 - atomicapp.utils - DEBUG - Recursively removing directory: /tmp/nulecule-cceaa7b8-bedd-11e5-a09b-3402868080db
2016-01-19 18:52:34,036 - atomicapp.nulecule.container - DEBUG - Removing Docker container: /usr/bin/docker rm -f b2251700ecc32b8cb10d01ce29e92034ac3f10a4f225b957c6f0d87e0220d5cf
b2251700ecc32b8cb10d01ce29e92034ac3f10a4f225b957c6f0d87e0220d5cf
2016-01-19 18:52:34,099 - atomicapp.plugin - DEBUG - Loading providers from /opt/atomicapp/atomicapp/providers
2016-01-19 18:52:34,101 - atomicapp.plugin - DEBUG - Loading providers from /opt/atomicapp/atomicapp/providers
2016-01-19 18:52:34,102 - atomicapp.plugin - DEBUG - Loading providers from /opt/atomicapp/atomicapp/providers
db_user (Database User): 

@dustymabe
Copy link
Contributor

@cdrage can you give the full output?

@rtnpro
Copy link
Contributor Author

rtnpro commented Jan 20, 2016

@cdrage which tests are you referring to?

@rtnpro rtnpro force-pushed the fix-answers-namespace-conflicts branch from a42df1f to dfbc912 Compare February 8, 2016 12:18
@rtnpro
Copy link
Contributor Author

rtnpro commented Feb 8, 2016

@cdrage I was able to reproduce the above scenario you had reported. It was a bug due to not passing component namespace when loading an external Nulecule application from path: https://github.com/projectatomic/atomicapp/pull/511/files#diff-4658e8bb2f390facf76690d16cd7d459R344.

@cdrage
Copy link
Member

cdrage commented Apr 19, 2016

Hey @rtnpro were you able to fix this? :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants