Skip to content

Update e2e pipeline #1202

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Nov 11, 2020
Merged

Update e2e pipeline #1202

merged 9 commits into from
Nov 11, 2020

Conversation

sdudoladov
Copy link
Member

@sdudoladov sdudoladov commented Nov 9, 2020 •
edited
Loading

  • remove confusing version change message when the version has not actually changed
  • fix a few important flake8 violations : incorrect comparison to None etc.
  • log ApiException for meaningful debugging
  • figure out where the operator pod actually gets deleted for config update
  • document how to fix
ERROR: no nodes found for cluster "postgres-operator-e2e-tests"

during e2e test runs

  • rename namespace to test_namespace
  • document how to use kubectl to connect to postgres-operator-e2e-tests . That currently does not work despite kubectl reporting success on the set-context operation with --kubeconfig=/tmp/kind-config-postgres-operator-e2e-tests being used
  • remove the dependencies between test_multinamespace_support on one hand and test_scaling and test_deletion on the other

@sdudoladov sdudoladov self-assigned this Nov 9, 2020
@sdudoladov sdudoladov mentioned this pull request Nov 11, 2020
Merged
@sdudoladov
Copy link
Member Author

👍

1 similar comment
@Jan-M
Copy link
Member

Jan-M commented Nov 11, 2020

👍

@sdudoladov sdudoladov merged commit e779eab into master Nov 11, 2020
PetterSa pushed a commit to PetterSa/postgres-operator that referenced this pull request Nov 19, 2020
@sdudoladov @PetterSa
Update e2e pipeline (zalando#1202)
128104c 
* clean up after test_multi_namespace test

* see the PR description for complete list of changes

Co-authored-by: Sergey Dudoladov <sergey.dudoladov@zalando.de>
PetterSa added a commit to PetterSa/postgres-operator that referenced this pull request Nov 19, 2020
@PetterSa
Revert "Update e2e pipeline (zalando#1202)"
73b57d1 
This reverts commit 128104c.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@avaczi avaczi Awaiting requested review from avaczi

@CyberDem0n CyberDem0n Awaiting requested review from CyberDem0n

@erthalion erthalion Awaiting requested review from erthalion

@FxKu FxKu Awaiting requested review from FxKu FxKu is a code owner

@Jan-M Jan-M Awaiting requested review from Jan-M Jan-M is a code owner

@RafiaSabih RafiaSabih Awaiting requested review from RafiaSabih

Assignees

@sdudoladov sdudoladov

Labels

needs review zalando

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sdudoladov @Jan-M