Skip to content

Remove q query for VC.#32

Merged
FragmentedPacket merged 3 commits intonautobot:developfrom
FragmentedPacket:30-vc-replace-bug
Apr 8, 2021
Merged

Remove q query for VC.#32
FragmentedPacket merged 3 commits intonautobot:developfrom
FragmentedPacket:30-vc-replace-bug

Conversation

@FragmentedPacket
Copy link
Copy Markdown
Contributor

Fixes #30

jvanderaa
jvanderaa previously approved these changes Apr 8, 2021
@jvanderaa
Copy link
Copy Markdown
Contributor

@FragmentedPacket once the tests are passing confirmed that it will create them idempotently.

Screen Shot 2021-04-08 at 15 35 52

@jvanderaa
Copy link
Copy Markdown
Contributor

However state of absent is failing.

@FragmentedPacket
Copy link
Copy Markdown
Contributor Author

Ok. I'll check that once I'm back online

@jvanderaa
Copy link
Copy Markdown
Contributor

Ok, I checked against a B3 image. This is working properly. There may have been an issue where a space was also included as I re-ran the tests. So this is working with the latest versions.

@FragmentedPacket
Copy link
Copy Markdown
Contributor Author

Both present and absent?

@jvanderaa
Copy link
Copy Markdown
Contributor

Yes. present and absent.

@jvanderaa
Copy link
Copy Markdown
Contributor

So I think it was either the b1 instance, or the space that was on the one unit that had issues. That may be something else that we should look into eventually.

@FragmentedPacket
Copy link
Copy Markdown
Contributor Author

@jvanderaa Tests are passing now :)

@FragmentedPacket FragmentedPacket merged commit 71c8dba into nautobot:develop Apr 8, 2021
@FragmentedPacket FragmentedPacket deleted the 30-vc-replace-bug branch April 8, 2021 22:00
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.

Adding 'virtual_chassis' will overwrite entries with different names

2 participants