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

Updates for Marathon 1.9.109 #270

Merged
merged 9 commits into from
May 20, 2020
Merged

Conversation

missingcharacter
Copy link
Contributor

No description provided.

@dPeS
Copy link

dPeS commented May 14, 2020

+1 here, 0.12 don't work with 1.9.136
:
File "/home/pepto/.virtualenvs/scaler/lib/python3.6/site-packages/marathon/client.py", line 222, in list_apps response, MarathonApp, is_list=True, resource_name='apps') File "/home/pepto/.virtualenvs/scaler/lib/python3.6/site-packages/marathon/client.py", line 72, in _parse_response return [clazz.from_json(resource) for resource in target] File "/home/pepto/.virtualenvs/scaler/lib/python3.6/site-packages/marathon/client.py", line 72, in <listcomp> return [clazz.from_json(resource) for resource in target] File "/home/pepto/.virtualenvs/scaler/lib/python3.6/site-packages/marathon/models/base.py", line 43, in from_json return cls(**{to_snake_case(k): v for k, v in attributes.items()}) TypeError: __init__() got an unexpected keyword argument 'role'

@missingcharacter
Copy link
Contributor Author

@dPeS do you get the role error when using my changes or do you encounter any other errors?

I'd like to improve my code if I can.

self.app = app
self.apps = apps
self.app = assert_valid_path(app.lower())
self.apps = assert_valid_path(apps.lower())
Copy link
Contributor

Choose a reason for hiding this comment

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

Is apps expected to be a string you can validate on like this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe so, I tried different iterations and this was the one that worked for me. I admit I'm no python expert.

@solarkennedy
Copy link
Contributor

Can you update the readme and maybe the test matrix?
I don't actually expect the build to go green, but maybe?

@missingcharacter
Copy link
Contributor Author

@solarkennedy is this what you asked for?

@solarkennedy
Copy link
Contributor

Cool. Can you check the failing tests next? They look real.

@missingcharacter
Copy link
Contributor Author

@solarkennedy do you happen to have or now of a docker image that has all of these versions of mesos and marathon already baked in?

I'm trying to build the 14.04 image and keeps failing on ppa:webupd8team/java which seems to be discontinued

@solarkennedy
Copy link
Contributor

I'm not aware of such an image.

I would accept a PR to use a standard image:
https://hub.docker.com/r/mesosphere/marathon
Maybe running in "development mode"?

Even if that means we have to only test against 1 version.

I don't know if there is a replacement for that PPA, or maybe the docker image itself could be upgraded to 20.04?

@missingcharacter
Copy link
Contributor Author

@solarkennedy I'll try https://hub.docker.com/r/mesosphere/marathon and Ubuntu 20.04, I'll get back to you

@missingcharacter
Copy link
Contributor Author

missingcharacter commented May 20, 2020

@solarkennedy I fixed the issues with the tests, whenever I have time I'll make another PR to fix the tests and use https://hub.docker.com/r/mesosphere/marathon

@solarkennedy solarkennedy merged commit 34ea9db into thefactory:master May 20, 2020
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.

3 participants