Fix backup and restore test

New Pull Request

Conversation 0 Commits 41 Files Changed 16

@jonathan please review

I got this almost working yesterdat, but today, without any signifant changes in code, I’m getting error from Vultr while trying to create instance for restored bubble. I’ll try again the test tomorrow - it should pass all the way with these changes.

The only corners I needed to cut here are deploying automation (ansible role) scripts. But that is not directly related.
Changes in automation files made in this PR are critical for restoring to work properly, and should go to all servers, all users, all domains. Existing script for deployment (update_role) works for only 1 role, 1 domain and 1 user. And I couldn’t even start it up (JavaHeapSpace, so I guess I need to check some settings on my end for further testing of this updater script).

jonathan was assigned by kris 4 years ago

kris changed title from ~~WIP: (testing) Fix backup and restore test~~ to Fix backup and restore test 4 years ago

jonathan referenced this issue from a commit 4 years ago

Fix backup and restore test (#16) Lower time waiting for restore instance to start in test Show better message on Vultr start server failure Delete plan instead of network in test Merge branch 'master' into kris/fix_backup_restore_test_cont_2 # Conflicts: # bubble-server/src/test/resources/models/include/new_bubble.json # utils/cobbzilla-wizard Change and add awaiting URLs required in restore test Do not lock accounts on node restore Set username in test in a single place Add test for new debug call and test helpers Update lib Merge branch 'master' into kris/fix_backup_restore_test_cont_2 # Conflicts: # utils/cobbzilla-utils # utils/cobbzilla-wizard Remove MITM nat iptables entries from ansible setup Log admin port Change log format to more standard one Rename algo related ansible tag Add some more grace time for restore Add more info in log messages and some more logging Revert wrong meter tick reordering Fix error in bash var reference Fix response from debug echo call Update algo's meter tick pattern Add debug echo API call Fix name position within service task in ansible Fix required indent Use ansible tags to properly run post-restore tasks Revert "Reference install_type where required" This reverts commit 9184cd8113c59ccb32875bed788ed2f2f332c0a0. Reorder ticks appropriately Look for only non-promotional payment methods in new bubble script Reference install_type where required Rename algo ansible tasks uniquely Wait for bubble to stop for real in the test Remove not needed stored vars from test script Use grace period in await_url for new bubbles Set waiting for keys in test inbox Set global mapping to be global static Show full exception logs Add activation key for new bubble's first login in test Call new bubble's API before DNS list in test update libs Beautify code Fix DNS list API request URL in test Better DNS listing APIs usage Co-authored-by: Kristijan Mitrovic <kmitrovic@itekako.com> Reviewed-on: https://git.bubblev.org/bubblev/bubble/pulls/16

jonathan closed this pull request 4 years ago

jonathan deleted branch kris/fix_backup_restore_test_cont_2 4 years ago

The pull request has been merged as 6cd5e5c16a.

Sign in to join this conversation.

No reviewers

No Label

No Milestone

jonathan

1 Participants

Notifications

Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

#16 Fix backup and restore test