Skip to content

Implement add_space and get_spaces#467

Merged
jujubot merged 3 commits intojuju:masterfrom
gnuoy:issue/466
Feb 1, 2021
Merged

Implement add_space and get_spaces#467
jujubot merged 3 commits intojuju:masterfrom
gnuoy:issue/466

Conversation

@gnuoy
Copy link
Copy Markdown

@gnuoy gnuoy commented Jan 30, 2021

This PR implements add_space and get_spaces. Since spaces are not
implemented with the LXD provider there is not corresponding
functional test. Closes issue #466.

This PR implements add_space and get_spaces. Since spaces are not
implemented with the LXD provider there is not corresponding
functional test. Closes issue juju#466.
@jujubot
Copy link
Copy Markdown
Contributor

jujubot commented Jan 30, 2021

Can one of the admins verify this patch?

9 similar comments
@jujubot
Copy link
Copy Markdown
Contributor

jujubot commented Jan 30, 2021

Can one of the admins verify this patch?

@jujubot
Copy link
Copy Markdown
Contributor

jujubot commented Jan 30, 2021

Can one of the admins verify this patch?

@jujubot
Copy link
Copy Markdown
Contributor

jujubot commented Jan 30, 2021

Can one of the admins verify this patch?

@jujubot
Copy link
Copy Markdown
Contributor

jujubot commented Jan 30, 2021

Can one of the admins verify this patch?

@jujubot
Copy link
Copy Markdown
Contributor

jujubot commented Jan 30, 2021

Can one of the admins verify this patch?

@jujubot
Copy link
Copy Markdown
Contributor

jujubot commented Jan 30, 2021

Can one of the admins verify this patch?

@jujubot
Copy link
Copy Markdown
Contributor

jujubot commented Jan 30, 2021

Can one of the admins verify this patch?

@jujubot
Copy link
Copy Markdown
Contributor

jujubot commented Jan 30, 2021

Can one of the admins verify this patch?

@jujubot
Copy link
Copy Markdown
Contributor

jujubot commented Jan 30, 2021

Can one of the admins verify this patch?

Copy link
Copy Markdown
Member

@SimonRichardson SimonRichardson left a comment

Choose a reason for hiding this comment

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

@manadart can you check this as well 👍

Comment thread juju/client/connection.py Outdated
Copy link
Copy Markdown
Member

@SimonRichardson SimonRichardson left a comment

Choose a reason for hiding this comment

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

Let's good from my perspective.

Copy link
Copy Markdown
Member

@manadart manadart left a comment

Choose a reason for hiding this comment

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

Looks good.

@SimonRichardson
Copy link
Copy Markdown
Member

!!build!!

@SimonRichardson
Copy link
Copy Markdown
Member

@gnuoy can you fix the lint issues, then I can land it.

juju/tag.py:54:1: E302 expected 2 blank lines, found 1

@SimonRichardson
Copy link
Copy Markdown
Member

!!build!!

@SimonRichardson
Copy link
Copy Markdown
Member

$$merge$$

1 similar comment
@SimonRichardson
Copy link
Copy Markdown
Member

$$merge$$

@SimonRichardson
Copy link
Copy Markdown
Member

$$merge$$

Failed to build python 3.5, considering it's EOL (#468) we should remove.

@jujubot jujubot merged commit d696937 into juju:master Feb 1, 2021
@SimonRichardson SimonRichardson mentioned this pull request Mar 31, 2021
jujubot added a commit that referenced this pull request Mar 31, 2021
#481

Merge 2.9 into master

This is brings in 2.9-rc3 schema as well, so that we're dealing with
the latest changes.

Conflicts:
 _client3.py - just re-ran the `make client` as the code is autogenerated.

5e23162 (upstream/master) Merge pull request #480 from SimonRichardson/release-2.8.6
3e23b74 (origin/master, origin/HEAD, master) Merge pull request #479 from SimonRichardson/schema-update-2.8.10
7f96863 Merge pull request #478 from SimonRichardson/scale-typo
fd1438c Merge pull request #475 from achilleasa/provide-accessor-for-machine-hostname-field
1015f4d Expand wait_for_idle to support waiting for status (#473)
7829fd0 Merge pull request #471 from SimonRichardson/release-2.8.5
fd36c8f Merge pull request #470 from juju/johnsca/bundle-charm-files-and-wait
3d91d6b Merge pull request #469 from juju/johnsca/accept-charm-files
4e4e108 Merge pull request #465 from juju/johnsca/update-websockets/py39
d696937 Merge pull request #467 from gnuoy/issue/466
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.

4 participants