Jira PSAP-1288: NTO operand cleanup

[jmencak]

In preparation for running TuneD prior to kubelet, make things simpler and cleaner.

Notable changes:

• Even though there is currently no namespace collision with TuneD using in /var/lib/tuned, change this path to /var/lib/ocp-tuned
• Remove bin/run. While this means a little code duplication across Containerfiles, we no longer need to do anything special at run time. This should make things easier for the future.
• Do not inherit --enable-leader-election and --version NTO flags as they are not handled by subcommands anyway (yet)
• Remove openshift-tuned binary and use NTO subcommand instead.
• /var/lib/tuned/profiles-data is no longer used, remove it.
• Remove openshift-tuned PID file code. It is no longer used.
• Clean up after PR844

[openshift-ci]

@jmencak: GitHub didn't allow me to request PR reviews from the following users: jmencak.

Note that only openshift members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

In preparation for running TuneD prior to kubelet, make things simpler and cleaner.

Notable changes:

• Even though there is currently no namespace collision with TuneD using in/var/lib/tuned, change this path to /var/lib/ocp-tuned
• Remove bin/run. While this means a little code duplication across Containerfiles, we no longer need to do anything special at run time. This should make things easier for the future.
• Do not inherit --enable-leader-election and --version NTO flags as they are not handled by subcommands anyway (yet)
• Remove openshift-tuned binary and use NTO subcommand instead.
• /var/lib/tuned/profiles-data is no longer used, remove it.
• Remove openshift-tuned PID file code. It is no longer used.
• Clean up after PR844

/cc

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[openshift-ci-robot]

@jmencak: This pull request references PSAP-1288 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.16.0" version, but no target version was set.

In response to this:

In preparation for running TuneD prior to kubelet, make things simpler and cleaner.

Notable changes:

• Even though there is currently no namespace collision with TuneD using in/var/lib/tuned, change this path to /var/lib/ocp-tuned
• Remove bin/run. While this means a little code duplication across Containerfiles, we no longer need to do anything special at run time. This should make things easier for the future.
• Do not inherit --enable-leader-election and --version NTO flags as they are not handled by subcommands anyway (yet)
• Remove openshift-tuned binary and use NTO subcommand instead.
• /var/lib/tuned/profiles-data is no longer used, remove it.
• Remove openshift-tuned PID file code. It is no longer used.
• Clean up after PR844

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

[jmencak]

/cc @yanirq @MarSik @Tal-or

[openshift-ci-robot]

@jmencak: This pull request references PSAP-1288 which is a valid jira issue.

In response to this:

In preparation for running TuneD prior to kubelet, make things simpler and cleaner.

Notable changes:

• Even though there is currently no namespace collision with TuneD using in /var/lib/tuned, change this path to /var/lib/ocp-tuned
• Remove bin/run. While this means a little code duplication across Containerfiles, we no longer need to do anything special at run time. This should make things easier for the future.
• Do not inherit --enable-leader-election and --version NTO flags as they are not handled by subcommands anyway (yet)
• Remove openshift-tuned binary and use NTO subcommand instead.
• /var/lib/tuned/profiles-data is no longer used, remove it.
• Remove openshift-tuned PID file code. It is no longer used.
• Clean up after PR844

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

[openshift-ci]

@jmencak: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[yanirq]

Until when do we need to maintain Dockerfile.rhel8 ?

[jmencak]

Until when do we need to maintain Dockerfile.rhel8 ?

As long as 4.12 is supported, because it is the last one, where NTO uses RHEL8 as the base. I'd say we can safely drop it after January 17, 2025.

[MarSik]

This looks good to me. I will let Yanir issue the last lgtm as he had a question.

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jmencak, MarSik

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [MarSik,jmencak]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[yanirq]

This looks good to me. I will let Yanir issue the last lgtm as he had a question.

lgtm as well (although the bot already has interpreted the comment as lgtm)

[MarSik]

Heh, I thought you have to use the command form of lgtm. The bot is too smart it seems :)

[openshift-bot]

[ART PR BUILD NOTIFIER]

This PR has been included in build cluster-node-tuning-operator-container-v4.16.0-202402271510.p0.g11ca6b4.assembly.stream.el9 for distgit cluster-node-tuning-operator.
All builds following this will include this PR.