Skip to content

#3: Add postgresql 9.6.#15

Merged
reynoldsalec merged 5 commits intomainfrom
3-pgsql
Oct 26, 2021
Merged

#3: Add postgresql 9.6.#15
reynoldsalec merged 5 commits intomainfrom
3-pgsql

Conversation

@reynoldsalec
Copy link
Copy Markdown
Member

@reynoldsalec reynoldsalec commented Oct 21, 2021

Two questions:

  1. When I run lando ssh -s postgresql -c "id" | grep app I get...

uid=501(postgres) gid=20(dialout) groups=20(dialout),102(ssl-cert),103(postgres)

...which of course fails. Am I not understanding the goal of that test?

  1. The postgres DB doesn't have anything loaded into it, so when I run lando postgresql main -c "\dt" I receive "No relations found." Is that ok? Was assuming based on the mysql tests that we're just checking to make sure the command runs without an error, not that data exists (which it shouldn't).

@reynoldsalec reynoldsalec force-pushed the 3-pgsql branch 3 times, most recently from b5df70b to 87a5fb7 Compare October 21, 2021 04:43
@reynoldsalec reynoldsalec force-pushed the 3-pgsql branch 2 times, most recently from 0296082 to 110e338 Compare October 21, 2021 17:28
@reynoldsalec
Copy link
Copy Markdown
Member Author

reynoldsalec commented Oct 21, 2021

I'm getting intermittent failures on the Postgres tests, typically on trying to run lando postgresql main -c "\dt" (see https://github.com/lando/platformsh/runs/3967494464?check_suite_focus=true#step:10:32)

Not able to replicate locally and the service it fails on varies, so it seems truly intermittent. Maybe I need to introduce a wait?

LMK if you have ideas on that @pirog

@reynoldsalec reynoldsalec mentioned this pull request Oct 21, 2021
68 tasks
@pirog
Copy link
Copy Markdown
Member

pirog commented Oct 22, 2021

@reynoldsalec yah im guessing you are right on the wait. i think what id do for now is just introduce a sufficiently large sleep so that the tests pass.

then spin up an issue that flags this concern eg "postgres isnt always ready after lando starts" or something. im guessing what we will want to do there is introduce a healthcheck for postgres to ensure it is ready to go before lando reports start as complete. in the issue we should also include a task to remove the sleep from the tests.

@reynoldsalec
Copy link
Copy Markdown
Member Author

@reynoldsalec
Copy link
Copy Markdown
Member Author

Looks like adding in a health check to the platform-postgresql service worked; postgres tests are passing. Merging.

@reynoldsalec reynoldsalec merged commit 26740bd into main Oct 26, 2021
@reynoldsalec reynoldsalec deleted the 3-pgsql branch October 26, 2021 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants