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

Remove FAKE_INFOBLOX (2/3) #634

Merged
merged 1 commit into from
Oct 2, 2021
Merged

Remove FAKE_INFOBLOX (2/3) #634

merged 1 commit into from
Oct 2, 2021

Conversation

kuritka
Copy link
Collaborator

@kuritka kuritka commented Sep 30, 2021

This is a follow-up PR to #630 (1/3), do not review until #630 is merged.

Everything is now being mocked up, there is no need to distinguish between local and production infoblox.
Bool switch no longer makes sense, removing

Signed-off-by: kuritka [email protected]

Copy link
Member

@ytsarev ytsarev left a comment

Choose a reason for hiding this comment

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

Super happy to get rid of fake infoblox dead code, thanks :) It just looks like it should be rebased with #630 after its merge

@ytsarev
Copy link
Member

ytsarev commented Oct 2, 2021

Ah, you mentioned it, sorry :)

@kuritka kuritka requested a review from ytsarev October 2, 2021 13:15
This is a follow-up PR to #630 (1/3), do not review until #630 is merged

Everything is now being mocked up, there is no need to distinguish between local and production infoblox.
Bool switch no longer makes sense, removing

Signed-off-by: kuritka <[email protected]>
@kuritka kuritka force-pushed the remove-FAKE_INFOBLOX branch from 89e1408 to ff5b927 Compare October 2, 2021 13:17
@kuritka kuritka merged commit bc71033 into master Oct 2, 2021
@kuritka kuritka deleted the remove-FAKE_INFOBLOX branch October 2, 2021 22:22
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