Virtualization

[gridhead]

Added 13 pages of goodness. :)

Fixes #392

[gridhead]

PLEASE DO NOT MERGE YET

The section icon is broken and I do not know how to add one.

[gridhead]

Hey @pvalena, could you please see if you can do something about the broken image?

[jackorp]

Hey @pvalena, could you please see if you can do something about the broken image?

Hi, could you please create an issue at developer-portal/website describing the command you ran and the output?

I ran

$ docker run -it --rm -p4000:4000 -v "$PWD:/opt/developerportal/website/content:Z" quay.io/developer-portal/devel

successfully on this PR.

[gridhead]

Hi @jackorp,

Noting that the section that I am talking about here was newly introduced by me with this pull request. Here's the log.

podman run -it --rm -p 4000:4000 -v "$PWD:/opt/developerportal/website/content:Z" quay.io/developer-portal/devel

Configuration file: /opt/developerportal/website/_config.yml
            Source: /opt/developerportal/website
       Destination: /opt/developerportal/website/_site
 Incremental build: enabled
      Generating... 
       Git authors: Generating authors...
Traceback (most recent call last):
  File "/opt/developerportal/website/rss.py", line 23, in <module>
    feedparser._HTMLSanitizer.unacceptable_elements_with_end_tag.add('<div>')
AttributeError: module 'feedparser' has no attribute '_HTMLSanitizer'
                    done in 6.125 seconds.
 Auto-regeneration: enabled for '/opt/developerportal/website'
LiveReload address: http://0.0.0.0:35729
    Server address: http://0.0.0.0:4000/
  Server running... press ctrl-c to stop.
[2021-09-08 02:14:07] ERROR `/static/logo/virtualization.png' not found.
[2021-09-08 02:14:14] ERROR `/static/logo/virtualization.png' not found.
[2021-09-08 02:14:22] ERROR `/static/logo/virtualization.png' not found.

Here's a link to the ticket that I created there developer-portal/website#121.

[pvalena]

What confuses me that virtualization is a general term, or a category, rather than a specific tool. Ideas?

CC: @hhorak

[gridhead]

@pvalena, I do seem to have covered at least three tools from the virtualization but maybe we should have a major section to itself.

[pvalena]

Actually, looking once more at it, it looks like it fits into tools, as it quite describe the low-level tools one might use, when working with virtualization.

Thanks once more!

[pvalena]

Thanks for a such a broad PR!

I've written some suggestions, hopefully it's all straightforward.

I'd rearrange it a bit, and also change mainly:

• the code-block notations to command with $ and have the output follow right after the command in the same block
• trim some of the example output (the non-essential parts)
• describe what's being done, what's the aim of the command / sub-page, or expected result
• use sub-headers ## and sub-sub-headers ### instead of numbering where suitable.

Let me know if you have any questions.

[pvalena]

Could you also describe the outcomes / implications for a regular user? IOW what's the aim / expected result of this testing / comparison?

[pvalena]

This should have some more concise title like e.g.:

Suggested change

	# Playing around with various VGA providers
	# Setting up various VGA providers

[gridhead]

The reason why I would suggest it to stay the same is because most VGA providers are very experimental in nature and people would not want to use them in a critical environment. They would still want to play around with them as these are actively developed and might be stable someday. What do you think?

[pvalena]

Maybe that could be mentioned stand alone / explicitly - maybe with Experimental in the sub-title? (Instead of being "in between the lines", as many non-native speakers won't capture that information.)

[gridhead]

How about "Experimenting with various VGA providers" as a title?

[pvalena]

Sounds great!

[gridhead]

@pvalena,

Many thanks for the comprehensive review. I will line up the changes in a while and tag you back in when I am done making the suggested changes.

[gridhead]

@pvalena,

I made the requested changes. Do take a look. :)

[jackorp]

LGTM. Please also provide the source of the logo and license of the logo in the PR website#122, We can then merge both.

Thanks!

[gridhead]

@jackorp, developer-portal/website#122 is now merged.

Please see if this PR can be merged now.

[jackorp]

LGTM, let's merge it.

[pvalena]

@t0xic0der Thanks for the good work!

Let's polish this in staging:
- https://developer.stg.fedoraproject.org/tools/virtualization/about.html

[gridhead]

Sure @pvalena. I am proofreading the contents right now. What else needs to be added?