Skip to content

Conversation

@hanwen-cluster
Copy link
Contributor

@hanwen-cluster hanwen-cluster commented Mar 21, 2025

This PR is a followup of #2919

  1. Only cleanup resolv conf during official AMI build. In the future, we will evaluate to apply this improvement to all AMI builds.
  2. Also clean up /run/systemd/resolve/resolv.conf . This file exists on Ubuntu

Tests

  • AMI builds are successful

References

Checklist

  • Make sure you are pointing to the right branch.
  • If you're creating a patch for a branch other than develop add the branch name as prefix in the PR title (e.g. [release-3.6]).
  • Check all commits' messages are clear, describing what and why vs how.
  • Make sure to have added unit tests or integration tests to cover the new/modified code.
  • Check if documentation is impacted by this change.

Please review the guidelines for contributing and Pull Request Instructions.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

1. Only cleanup resolv conf during official AMI build. In the future, we will evaluate to apply this improvement to all AMI builds.
2. Also clean up `/run/systemd/resolve/resolv.conf `. This file exists on Ubuntu

Signed-off-by: Hanwen <hanwenli@amazon.com>
rm -f /etc/sysconfig/network-scripts/ifcfg-eth0
fi

# Clean resolv.conf if it's not managed by system
Copy link
Contributor

Choose a reason for hiding this comment

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

could you please align this comment to the current strategy and put a short explanation, so that we will not forget i future?

@hanwen-cluster hanwen-cluster merged commit 1c425ba into aws:release-3.13 Mar 21, 2025
38 of 45 checks passed
hanwen-cluster added a commit to hanwen-cluster/aws-parallelcluster that referenced this pull request Mar 24, 2025
The boot speed of Rocky and RHEL were improved after aws/aws-parallelcluster-cookbook#2920

Signed-off-by: Hanwen <hanwenli@amazon.com>
hanwen-cluster added a commit to aws/aws-parallelcluster that referenced this pull request Mar 24, 2025
The boot speed of Rocky and RHEL were improved after aws/aws-parallelcluster-cookbook#2920

Signed-off-by: Hanwen <hanwenli@amazon.com>
himani2411 pushed a commit to himani2411/aws-parallelcluster that referenced this pull request Apr 2, 2025
The boot speed of Rocky and RHEL were improved after aws/aws-parallelcluster-cookbook#2920

Signed-off-by: Hanwen <hanwenli@amazon.com>
himani2411 pushed a commit to aws/aws-parallelcluster that referenced this pull request Apr 3, 2025
The boot speed of Rocky and RHEL were improved after aws/aws-parallelcluster-cookbook#2920

Signed-off-by: Hanwen <hanwenli@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants