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

Paris F2F Relese 1.0 Cleanup #101

Merged
merged 27 commits into from
Jul 16, 2019
Merged

Paris F2F Relese 1.0 Cleanup #101

merged 27 commits into from
Jul 16, 2019

Conversation

rabi-abdel
Copy link
Collaborator

@rabi-abdel rabi-abdel commented Jul 14, 2019

So far:

incorporate @markshostak pull request #100
@rabiabdel rabiabdel changed the title Overall Paris F2F Cleanup Paris F2F Relese 1.0 Cleanup Jul 14, 2019
@rabiabdel rabiabdel mentioned this pull request Jul 14, 2019
@xavier-grall
Copy link
Collaborator

As requested by Rabi, our opinion (Karine and me, ie Orange) is that the following elements are not related to hw profile, but to sw profile or infra abstraction, and thus, they should not appear on that diagram (and even not in chapter 6):

  • oversubscription level
  • cpu pinning
  • hugepages
    The main rationale is that their management (activation, configuration, handling) requires at least an OS to be installed on the hw layer.
    With the same rationale, DPDK and NIC bonding, which are also mentioned in chapter 6, are not elements of the hw profile either.

@rabi-abdel
Copy link
Collaborator Author

Hi @xavier-grall I agree. I will modify it accordingly today as part of the overall cleanup. Thanks

Rabi Abdel added 3 commits July 15, 2019 12:54
also includes moving mappint tables to Chapter 4 for consistency.
@pgoyal01
Copy link
Collaborator

pgoyal01 commented Jul 15, 2019 via email

Rabi Abdel and others added 2 commits July 15, 2019 14:51
some cleanup and integration of 	@ASawwaf contribnution into Chapter 2.
@xavier-grall
Copy link
Collaborator

Pankaj,
Of course, that information must be included to correctly instanciate a VM (especially for choosing the hosting compute node), and they are included in the configuration of NFVI software and/or VIM configurations, but not of NFVI hardware (and thus, they are not related to chapter 6).

@xavier-grall
Copy link
Collaborator

Pankaj,
Of course, that information must be included to correctly instanciate a VM (especially for choosing the hosting compute node), and they are included in the configuration of NFVI software and/or VIM, but not of NFVI hardware (and thus, they are not related to chapter 6).

@bfcohen
Copy link
Collaborator

bfcohen commented Jul 15, 2019

We do need to be careful about what is part of the HW layer and what is part of the SW layer. Agree that what was called out is in fact part of the infrastructure software layer and as such should be moved. However this does bring up an issue of where to put in BIOS, IPMI and other firmware settings that more directly interface with the actual hardware. I would assume since those are typically supported by the HW vendors (Intel, Dell, etc.), these need to be in the HW section.

@rabiabdel
Copy link
Collaborator

@cntt-n great discussion everyone. i will try to incorporate those opinions for the discussion tomorrow.

@rabiabdel
Copy link
Collaborator

@ASawwaf, All can you check Chapter 2 modification and let me know if this looks good?

@xavier-grall
Copy link
Collaborator

@bfcohen
I agree: in my opinion, the sw layer "begins" with host OS, while the hw layer "ends" with BIOS (so IPMI and other firmwares are in the hw layer)

Rabi Abdel and others added 3 commits July 15, 2019 17:08
VA + cosmetic edits
consistent with rest of the document
This was referenced Jul 15, 2019
@pgoyal01
Copy link
Collaborator

pgoyal01 commented Jul 15, 2019 via email

@rabiabdel
Copy link
Collaborator

Really impressed with the quality of all Chapters :) Great work everyone @cntt-n. Talk to you tomorrow.

Copy link
Collaborator

@rabiabdel rabiabdel left a comment

Choose a reason for hiding this comment

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

Approved on 16th July 2019

@rabiabdel rabiabdel merged commit 6c4b750 into master Jul 16, 2019
@rabiabdel rabiabdel deleted the rabi-abdel-paris-f2f-cleanup branch July 16, 2019 12:41
@rabi-abdel rabi-abdel added the Archive Archive Item label Feb 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Archive Archive Item
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants