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

Switch ID and Lan ID constantly changing for ExternalDistributedVirtualSwitch #664

Open
ahrechushkin opened this issue Apr 19, 2024 · 7 comments
Assignees

Comments

@ahrechushkin
Copy link
Contributor

Switch ID and Lan ID constantly changing for ExternalDistributedVirtualSwitch

 #<ManageIQ::Providers::Ovirt::InfraManager::DistributedVirtualSwitch:0x00007ff649455c08
  id: 4,
  name: "awesomenet",
  ports: nil,
  created_on: Fri, 19 Apr 2024 11:32:14.906231000 UTC +00:00,
  updated_on: Fri, 19 Apr 2024 11:32:14.906231000 UTC +00:00,
  uid_ems: "63031fe4-518b-4e96-a248-ff85078f3d26",
  allow_promiscuous: nil,
  forged_transmits: nil,
  mac_changes: nil,
  switch_uuid: nil,
  shared: nil,
  mtu: nil,
  ems_id: 3,
  type: "ManageIQ::Providers::Ovirt::InfraManager::DistributedVirtualSwitch",
  health_state: nil,
  power_state: nil,
  host_id: nil>,
 #<ManageIQ::Providers::Ovirt::InfraManager::ExternalDistributedVirtualSwitch:0x00007ff6494558e8
  id: 9,
  name: "external_awesomenet",
  ports: nil,
  created_on: Fri, 19 Apr 2024 11:33:57.168437000 UTC +00:00,
  updated_on: Fri, 19 Apr 2024 11:33:57.168437000 UTC +00:00,
  uid_ems: "4d4cdbea-448a-4fe8-a691-e4bb633ce907",
  allow_promiscuous: nil,
  forged_transmits: nil,
  mac_changes: nil,
  switch_uuid: nil,
  shared: nil,
  mtu: nil,
  ems_id: 3,
  type: "ManageIQ::Providers::Ovirt::InfraManager::ExternalDistributedVirtualSwitch",
  health_state: nil,
  power_state: nil,
  host_id: nil>,

 #<ManageIQ::Providers::Ovirt::InfraManager::DistributedVirtualSwitch:0x00007ff641537b38
  id: 4,
  name: "awesomenet",
  ports: nil,
  created_on: Fri, 19 Apr 2024 11:32:14.906231000 UTC +00:00,
  updated_on: Fri, 19 Apr 2024 11:32:14.906231000 UTC +00:00,
  uid_ems: "63031fe4-518b-4e96-a248-ff85078f3d26",
  allow_promiscuous: nil,
  forged_transmits: nil,
  mac_changes: nil,
  switch_uuid: nil,
  shared: nil,
  mtu: nil,
  ems_id: 3,
  type: "ManageIQ::Providers::Ovirt::InfraManager::DistributedVirtualSwitch",
  health_state: nil,
  power_state: nil,
  host_id: nil>,
 #<ManageIQ::Providers::Ovirt::InfraManager::ExternalDistributedVirtualSwitch:0x00007ff6415379d0
  id: 11,
  name: "external_awesomenet",
  ports: nil,
  created_on: Fri, 19 Apr 2024 11:34:07.989047000 UTC +00:00,
  updated_on: Fri, 19 Apr 2024 11:34:07.989047000 UTC +00:00,
  uid_ems: "4d4cdbea-448a-4fe8-a691-e4bb633ce907",
  allow_promiscuous: nil,
  forged_transmits: nil,
  mac_changes: nil,
  switch_uuid: nil,
  shared: nil,
  mtu: nil,
  ems_id: 3,
  type: "ManageIQ::Providers::Ovirt::InfraManager::ExternalDistributedVirtualSwitch",
  health_state: nil,
  power_state: nil,
  host_id: nil>,

The logs here from latest image petrosian-1, also i tested in on jansa

Is it normal behaviour ? If yes - why?

@ahrechushkin
Copy link
Contributor Author

@agrare can you check it?

@agrare agrare self-assigned this Apr 19, 2024
@agrare
Copy link
Member

agrare commented Apr 22, 2024

Hey @ahrechushkin no this is not expected you should be able to refresh the provider multiple times and not have existing inventory be deleted and new items created.

Typically this is an issue with the query scope during a targeted refresh not finding the existing records thus creating new ones. I'll see if I can reproduce locally first otherwise I'll ask you to provide some logs so we can see when these are being deleted&created.

@ahrechushkin
Copy link
Contributor Author

Hello! @agrare
Seems like the problem with identification distributed virtual switches and external distributed virtual switches
Refresher delete external distributed virtual switches while refresh distributed virtual switches

@ahrechushkin
Copy link
Contributor Author

Probably we need to create model ManageIQ::Providers::InfraManager::ExternalDistributedVirtualSwitch (the same as ManageIQ::Providers::InfraManager::DistributedVirtualSwitch and inherit ManageIQ::Providers::Ovirt::InfraManager::ExternalDistributedVirtualSwitch from a newly create model instead of ManageIQ::Providers::InfraManager::DistributedVirtualSwitch to fully implement STI)

I hope it should work pretty

Im going to try on my own dev stand

@ahrechushkin
Copy link
Contributor Author

I tried on local stand and it works

ManageIQ/manageiq#23000
#665

@ahrechushkin
Copy link
Contributor Author

@agrare hi! do you have any news related to the issue?

@miq-bot miq-bot added the stale label Aug 26, 2024
@miq-bot
Copy link
Member

miq-bot commented Aug 26, 2024

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants