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

Implement CockroachDB preset #95

Merged
merged 3 commits into from
Jan 30, 2021
Merged

Implement CockroachDB preset #95

merged 3 commits into from
Jan 30, 2021

Conversation

orlangure
Copy link
Owner

@orlangure orlangure commented Jan 8, 2021

Implements #92.

This PR adds a new preset - cockroachdb. It includes basic initial state setup like in other database presets, except for an user/password setup. root user without a password is used this preset.

To connect to this DB root user without a password should be used with regular go postgres driver (see tests for an example).

Another couple of changes that are included here:

  • improving container cleanup in gnomock server
  • limiting memory of elasticsearch containers

@orlangure orlangure added the enhancement New feature or request label Jan 8, 2021
@orlangure orlangure self-assigned this Jan 8, 2021
@orlangure orlangure linked an issue Jan 8, 2021 that may be closed by this pull request
@codecov-io
Copy link

codecov-io commented Jan 8, 2021

Codecov Report

Merging #95 (34e34dc) into master (e311160) will decrease coverage by 60.80%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #95       +/-   ##
===========================================
- Coverage   80.98%   20.18%   -60.81%     
===========================================
  Files          42       43        +1     
  Lines        1678     1734       +56     
===========================================
- Hits         1359      350     -1009     
- Misses        165     1326     +1161     
+ Partials      154       58       -96     
Impacted Files Coverage Δ
preset/cockroachdb/options.go 0.00% <0.00%> (ø)
preset/cockroachdb/preset.go 0.00% <0.00%> (ø)
preset/postgres/options.go 0.00% <ø> (-100.00%) ⬇️
preset/postgres/preset.go 0.00% <ø> (-79.11%) ⬇️
parallel.go 0.00% <0.00%> (-100.00%) ⬇️
preset/k3s/options.go 0.00% <0.00%> (-100.00%) ⬇️
preset/kafka/options.go 0.00% <0.00%> (-100.00%) ⬇️
preset/mongo/options.go 0.00% <0.00%> (-100.00%) ⬇️
preset/mssql/options.go 0.00% <0.00%> (-100.00%) ⬇️
preset/mysql/options.go 0.00% <0.00%> (-100.00%) ⬇️
... and 34 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e311160...34e34dc. Read the comment docs.

@orlangure orlangure marked this pull request as ready for review January 9, 2021 09:39
@orlangure orlangure force-pushed the cockroachdb branch 2 times, most recently from 0fea6c8 to 99dbdcb Compare January 14, 2021 20:24
@orlangure orlangure force-pushed the cockroachdb branch 4 times, most recently from d9eddcc to 34e34dc Compare January 30, 2021 15:58
With this commit, gnomock container will stop trying to create
containers when incoming requests are canceled. It will allow to trigger
cleanup in more cases than before.

Also, this commit removes "debug" flag from some tests. Debug feature
has its own tests, so it's not required that other tests use it.
Without this change, when elasticsearch containers were executed
together with some other ones, sometimes they were killed (out of
memory). Now these containers are limited to 256m which should be enough
for tests.
@orlangure orlangure merged commit 2fd9e80 into master Jan 30, 2021
@orlangure orlangure deleted the cockroachdb branch January 30, 2021 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CockroachDB preset
2 participants