You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2021/12/07 00:25:49 UTC

[GitHub] [druid] jansonjc opened a new pull request #11876: Environment file changes to make it work with Docker Compose V2

jansonjc opened a new pull request #11876:
URL: https://github.com/apache/druid/pull/11876


   Fixes #11875.
   
   ### Description
   
   There is a breaking change in Docker Compose V2 that throws an error while parsing the environment file. One solution is the wrap the environment variables in double quote (and replacing existing double quotes to single quotes). 
   
   This PR has:
   - [X] been self-reviewed.
   - [X] been tested in both V2 and V1.x Docker Compose.
   


-- 
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: commits-unsubscribe@druid.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org

[GitHub] [druid] FrankChen021 commented on pull request #11876: Environment file changes to make it work with Docker Compose V2

Posted by GitBox <gi...@apache.org>.
FrankChen021 commented on pull request #11876:
URL: https://github.com/apache/druid/pull/11876#issuecomment-961928897






-- 
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: commits-unsubscribe@druid.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org

[GitHub] [druid] clintropolis closed pull request #11876: Environment file changes to make it work with Docker Compose V2

Posted by GitBox <gi...@apache.org>.
clintropolis closed pull request #11876:
URL: https://github.com/apache/druid/pull/11876


   


-- 
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: commits-unsubscribe@druid.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org

[GitHub] [druid] bjornjorgensen commented on pull request #11876: Environment file changes to make it work with Docker Compose V2

Posted by GitBox <gi...@apache.org>.
bjornjorgensen commented on pull request #11876:
URL: https://github.com/apache/druid/pull/11876#issuecomment-968370749


   Hi, I have some problems with this fix.
   
   2021-11-14T21:47:09,020 INFO [main] org.apache.druid.guice.JsonConfigurator - Unable to parse [druid.extensions.loadList]=[['druid-histogram', 'druid-datasketches', 'druid-lookups-cached-global', 'postgresql-metadata-storage']] as a json object, using as is.
   coordinator    | com.fasterxml.jackson.core.JsonParseExce
   [druid.txt](https://github.com/apache/druid/files/7534759/druid.txt)
   ption: Unexpected character (''' (code 39)): expected a valid value (JSON String, Number, Array, Object or token 'null', 'true' or 'false')
   coordinator    |  at [Source: (String)"['druid-histogram', 'druid-datasketches', 'druid-lookups-cached-global', 'postgresql-metadata-storage']"; line: 1, column: 3]
   
   


-- 
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: commits-unsubscribe@druid.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org

[GitHub] [druid] FrankChen021 commented on pull request #11876: Environment file changes to make it work with Docker Compose V2

Posted by GitBox <gi...@apache.org>.
FrankChen021 commented on pull request #11876:
URL: https://github.com/apache/druid/pull/11876#issuecomment-961928897


   Thanks for opening the PR. Would you please also fix the environment files under integration-test?  See #11859


-- 
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: commits-unsubscribe@druid.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org