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

[C 6.2] Time scheduling resources (same vlan) fail #546

Closed
juztas opened this issue Jun 5, 2024 · 7 comments
Closed

[C 6.2] Time scheduling resources (same vlan) fail #546

juztas opened this issue Jun 5, 2024 · 7 comments
Labels
bug Something isn't working
Milestone

Comments

@juztas
Copy link
Collaborator

juztas commented Jun 5, 2024

See comment here: https://github.com/esnet/StackV/issues/1141#issuecomment-2149994151

Looks like once second is added - it does either clean first/or second ports and switching subnet info.

@juztas juztas added the bug Something isn't working label Jun 5, 2024
@juztas juztas added this to the 1.4.9 Release milestone Jun 5, 2024
@juztas
Copy link
Collaborator Author

juztas commented Jun 5, 2024

So the problem is the following (not a quick fix):

  1. Request (1) for A to B submitted to start at +15m, finish +15m; lets say vlan 3600;
  2. SiteRM keeps track of all unique URIs for switchPorts and serverPorts (as they need to be unique)
  3. Agent's and Switch retrieval will use those URIs, once reporting information back inside the model. So any vlan on a specific switch/port or server and vlan3600 interface (ip, mac, etc) will use that unique URI.
  4. Request (2) for A to B submitted to start at +60m, finish +15m; lets say same vlan 3600; In terms of timing - they do not overlap and can run.
  5. Once Request (2) is accepted/added into the model - SiteRM parser get's confused and flip-flop things around. So it get's uniq uri for dellos9_s0/hundredGigE_1-32 to be urn:ogf:network:ultralight-dev.org:2013:dellos9_s0:hundredGigE_1-32:conn+3db5332f-62a1-4e15-8fbe-94eb4dfdcf07:vt+l2-policy-Connection_1:vlanport+3630 from Request (1), but continue parsing, it also get's urn:ogf:network:ultralight-dev.org:2013:dellos9_s0:hundredGigE_1-32:conn+3db5332f-62a1-4e15-8fbe-94eb4dfdcf07:vt+l2-policy-Connection_1:vlanport+3630.

@juztas
Copy link
Collaborator Author

juztas commented Jun 5, 2024

#547

@juztas
Copy link
Collaborator Author

juztas commented Jun 5, 2024

2 tests ongoing right now. One already started and it did not remove another delta information. Now it will use timing to identify which URIs are active; Looks to be working perfectly

@juztas juztas closed this as completed Jun 5, 2024
@xi-yang xi-yang reopened this Jun 11, 2024
@xi-yang
Copy link

xi-yang commented Jun 11, 2024

One problem remains. Both instances got READY as scheduled. When the first schedule is activated, the second switchingSubnet lost its two VLAN sub interfaces. This resulted in manifest and viz rendering problem.

The first:
Screenshot 2024-06-11 at 9 56 42 AM

The second:
Screenshot 2024-06-11 at 9 57 29 AM

@juztas
Copy link
Collaborator Author

juztas commented Jun 11, 2024

This was tested under 'test' release, deployed only on the ultralight-dev instance. I am working on a few small fixes which I expect to be done in the next hour.

Moving forward, see #557, I will have a separate instance (total 3) just for a development release. (that is for the next development cycle)

I will let you know once this fix is deployed on SoCal.

@xi-yang
Copy link

xi-yang commented Jun 11, 2024

This was tested under 'test' release, deployed only on the ultralight-dev instance. I am working on a few small fixes which I expect to be done in the next hour.

Moving forward, see #557, I will have a separate instance (total 3) just for a development release. (that is for the next development cycle)

I will let you know once this fix is deployed on SoCal.

Sounds good. Let's keep this issue open. I will check it off from the StackV issue.

@juztas
Copy link
Collaborator Author

juztas commented Jun 12, 2024

New tests succeeded: 180b719a-18a2-4b9f-993c-3b6d72a703cb and 7b412894-da37-4927-9125-9df44823ca02.

Closing. Lets reopen if this happens again

@juztas juztas closed this as completed Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants