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

feat: add oracle linux #670

Merged
merged 8 commits into from
Sep 22, 2023

Conversation

virtualistic
Copy link
Contributor

@virtualistic virtualistic commented Sep 17, 2023

Summary of Pull Request

As I requested here:
#659 and inspired by your suggestion, created a new build for Oracle Linux 8. Had to change the menu a bit to "make room" for the Oracle 8 template at position 19. (I skipped position 18 for Oracle 9 in the future). I was able to successfully build a new Oracle 8 template but hopefully you have some time to test and verify this.

Type of Pull Request

  • This is a bugfix. type/bug
  • This is an enhancement or feature. type/feature or type/enhancement
  • This is a documentation update. type/docs
  • This is a refactoring update. type/refactor
  • This is a chore. type/chore
  • This is something else.
    Please describe:

Related to Existing Issues

Closes #671

Test and Documentation Coverage

  • Tests have been completed.
  • Documentation has been added or updated.

Breaking Changes?

  • Yes, there are breaking changes.
  • No, there are no breaking changes.

@virtualistic virtualistic requested a review from a team as a code owner September 17, 2023 10:45
@vmwclabot
Copy link

@virtualistic, you must sign our contributor license agreement before your changes are merged. Click here to sign the agreement. If you are a VMware employee, read this for further instruction.

@vmwclabot
Copy link

@virtualistic, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed.

Copy link
Contributor

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

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

The build.yaml needs to be updated. This is used to generated the build.sh with gomplate.

The README.md would also need an update.

@tenthirtyam tenthirtyam changed the title Oel8 template feat: add oracle linux 8 Sep 17, 2023
@tenthirtyam tenthirtyam added this to the Backlog milestone Sep 17, 2023
@tenthirtyam tenthirtyam marked this pull request as draft September 17, 2023 14:25
@tenthirtyam
Copy link
Contributor

Marked as drafted until the CLA is approved.

@virtualistic
Copy link
Contributor Author

I added a new commit which added Oracle Linux as an entry to the README
Also added an entry for build.yaml

Copy link
Contributor

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

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

Hi @virtualistic -

Please run the following to generate the build.sh:

gomplate -c build.yaml -f build.tmpl -o build.sh

Additionally, in the README.md and update is needed to list the download information for each the distribution:

Ryan Johnson
Senior Staff Solutions Architect | Product Engineering @ VMware, Inc.

@virtualistic
Copy link
Contributor Author

Created a new commit that:

  • Rebuilt the build.sh using the command:
    gomplate -c build.yaml -f build.tmpl -o build.sh

  • Added the source for the Oracle ISO to the README

@tenthirtyam
Copy link
Contributor

Thanks! Once the CLA is approved I'll run the standard tests. I have a couple updates and pull requests that I have in my backlog for this side project but will prioritize as soon as possible.

Might I ask you to open an enhancement issue and link the pull request with "Closes #x" when time permits.

@virtualistic
Copy link
Contributor Author

Thanks! Once the CLA is approved I'll run the standard tests. I have a couple updates and pull requests that I have in my backlog for this side project but will prioritize as soon as possible.

Might I ask you to open an enhancement issue and link the pull request with "Closes #x" when time permits.

Sure, no problem. ( Enhancement request has a typo btw: "Please add a short decsription." )
When I open a new Enhancement Request, there are several mandatory fields:
Description
Use Case(s)
Potential Configuration
Do I type: "closes # 670" in each field or do you want that placed in a certain field?

@tenthirtyam
Copy link
Contributor

Closes is put here in the PR back to the enhancement issue.

thanks for the note on the issue form! :)

@virtualistic
Copy link
Contributor Author

closes #671

@virtualistic
Copy link
Contributor Author

virtualistic commented Sep 17, 2023

Closes is put here in the PR back to the enhancement issue.

thanks for the note on the issue form! :)

Most welcome!
Please note that the Oracle iso has an annoying mix of CaPs in its filename which I changed to all lowercase after I downloaded the iso and also used the lowercase name in the packer file that references the iso.
So 2 questions:

  1. Do you want me to use the original filename of the ISO in this pull request?

  2. I also changed the iso_path to iso/misc for my situation in the linux-oel.auto.pkrvars.hcl . Do you want me to change it to iso/linux/oracle for this pull request?

@tenthirtyam
Copy link
Contributor

Please use the original file name from the vendor and an oel or oracle path.

@vmwclabot
Copy link

@virtualistic, VMware has approved your signed contributor license agreement.

@tenthirtyam tenthirtyam marked this pull request as ready for review September 18, 2023 12:20
@tenthirtyam tenthirtyam dismissed their stale review September 20, 2023 22:14

Second review.

README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

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

Minor changes.

Would you consider adding OEL 9?

@tenthirtyam tenthirtyam added status/planned Planned status/awaitng-response Awaiting Response and removed status/under-review Under Review labels Sep 20, 2023
@virtualistic
Copy link
Contributor Author

virtualistic commented Sep 21, 2023

Minor changes.

Would you consider adding OEL 9?

Don't know if v9 will work because the latest vCenter v7 does not offer Oracle Linux 9 as a guest os .
(while my local v7 ESXi server does)
2023-09-21 21_29_16-vCenter-oracleinux-versions

@virtualistic
Copy link
Contributor Author

I agree with your suggested changes. Think it's best I edit my repo and create a new commit with the changes?
Or should I do a "signoff and commit suggestion" from the GitHub UI?

@tenthirtyam
Copy link
Contributor

I agree with your suggested changes. Think it's best I edit my repo and create a new commit with the changes?
Or should I do a "signoff and commit suggestion" from the GitHub UI?

"signoff and commit suggestion" is fine, but I generally just amend the last commit and force push.

@tenthirtyam
Copy link
Contributor

Don't know if v9 will work because the latest vCenter v7 does not offer Oracle Linux 9 as a guest os . (while my local v7 ESXi server does).

Use other5xLinux64Guest. 😃

@virtualistic
Copy link
Contributor Author

virtualistic commented Sep 22, 2023

Ok, I changed all the requested files and did a :

git add -A
git commit --amend --no-edit
git push -f

Will have look if it's easy to add Oracle 9 now using my own Oracle 8 files 😄

@virtualistic
Copy link
Contributor Author

Ok, pushed a new commit with an Oracle 9 Template.
2023-09-22 13_56_08-oel9

@tenthirtyam
Copy link
Contributor

Awesome! thank you!!

tenthirtyam
tenthirtyam previously approved these changes Sep 22, 2023
Copy link
Contributor

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

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

LGTM!

@tenthirtyam tenthirtyam changed the title feat: add oracle linux 8 feat: add oracle linux Sep 22, 2023
Update `CHANGELOG.md`

Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
Signed-off-by: Ryan Johnson <johnsonryan@vmware.com>
Copy link
Contributor

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

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

LGTM!

@tenthirtyam tenthirtyam merged commit b2202c0 into vmware-samples:develop Sep 22, 2023
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for Oracle Enterprise Linux
3 participants