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

Upgrade vagrant config to Ubuntu 18.04 and fix mongo 3 incompatibility #519

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mduggan
Copy link
Contributor

@mduggan mduggan commented Oct 21, 2019

This should fix the ability to use vagrant on a fresh checkout. It upgrades to Bionic (18.04).

Travis is showing this as not working, but that's due to the existing problem there - the tests pass when I run them from the local vagrant image, with some minor warnings:

vagrant@vagrant:/vagrant$ python runtests.py
../vagrant/tapiriik/services/interchange.py:151: DeprecationWarning: insert is deprecated. Use insert_one or insert_many instead.
  cachedb.tz_cache.insert(cachedTzData)
Handling streams ['path', 'heart_rate', 'calories']
......................./vagrant/tapiriik/testing/testtools.py:65: DeprecationWarning: insert is deprecated. Use insert_one or insert_many instead.
  db.test.insert({"asd": "asdd"})
		mockA not configured
.		Excluded mockA
./vagrant/tapiriik/auth/__init__.py:206: DeprecationWarning: update is deprecated. Use replace_one, update_one or update_many instead.
  db.users.update({"_id": user["_id"]}, {"$set": {"FlowExceptions": user["FlowExceptions"]}})
		Flow exception for mockB
.		Flow exception for mockA
		Flow exception for mockB
.		Flow exception for mockA
		Flow exception for mockB
		Flow exception for mockB
.		Flow exception for mockA
		Flow exception for mockB
.		Flow exception for mockA
.		Flow exception for mockC
		Flow exception for mockC
......	...mockB doesn't support type XCSkiing
	...mockA doesn't support type Cycling
	...mockA doesn't support type Cycling
.
----------------------------------------------------------------------
Ran 39 tests in 0.405s

OK
sys:1: ResourceWarning: unclosed file <_io.TextIOWrapper name='/tmp/gc_rate.0.0.0.0.lock' mode='r+' encoding='UTF-8'>
sys:1: ResourceWarning: unclosed file <_io.TextIOWrapper name='/tmp/m_rate.0.0.0.0.lock' mode='r+' encoding='UTF-8'>
sys:1: ResourceWarning: unclosed <socket.socket fd=9, family=AddressFamily.AF_INET, type=SocketKind.SOCK_STREAM, proto=6, laddr=('127.0.0.1', 57890), raddr=('127.0.0.1', 5672)>

I tried to also update .travis.yml config (like #502), but I couldn't get it working - something about the newer ubuntus in travis having an IPV6 localhost record (eg, https://travis-ci.org/mduggan/tapiriik/builds/600559948)

(fixes #491)

@@ -178,7 +178,10 @@ def cache_writeback():
cachedb.dropbox_cache.save(cache)
else:
insert_result = cachedb.dropbox_cache.insert(cache)
cache["_id"] = insert_result.inserted_id
if hasattr(insert_result, 'inserted_id'):
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was the only incompatibility I found with the latest version of things - although I only connected dropbox and runkeeper, but syncing between them worked.

@klofrau
Copy link

klofrau commented Jan 30, 2020

Works for me, thanks a lot!

Copy link

@klofrau klofrau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fresh setup works with these changes.

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.

Vagrant provisioning fails
2 participants