lordgamez commented on a change in pull request #1121:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1121#discussion_r696507787



##########
File path: docker/test/integration/minifi/core/PostgreSQLServerContainer.py
##########
@@ -0,0 +1,20 @@
+from .Container import Container
+
+
+class PostgreSQLServerContainer(Container):
+    def __init__(self, name, vols, network, image_store):
+        super().__init__(name, 'postgresql-server', vols, network, image_store)
+
+    def get_startup_finished_log_entry(self):
+        return "database system is ready to accept connections"
+
+    def deploy(self):
+        if not self.set_deployed():
+            return
+
+        self.docker_container = self.client.containers.run(
+            self.image_store.get_image(self.get_engine()),
+            detach=True,
+            name='postgresql-server',
+            network=self.network.name,
+            environment=["POSTGRES_PASSWORD=password"])

Review comment:
       In the connectionstring of the SQL processors we set the SQL server 
hostname to the name of the Postgres container and docker's DNS resolver takes 
care to know where to route the messages in the docker network.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to