Skip to content

Revert "fix: install node for data engineering jenkins" - #364

Merged
amits-2u merged 1 commit into
masterfrom
revert-363-rahul/install-node-for-de-jenkins-health-check
Aug 4, 2026
Merged

Revert "fix: install node for data engineering jenkins"#364
amits-2u merged 1 commit into
masterfrom
revert-363-rahul/install-node-for-de-jenkins-health-check

Conversation

@rahulkanneri-2u

Copy link
Copy Markdown
Contributor

Reverts #363

Copilot AI lite review requested due to automatic review settings August 4, 2026 07:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR reverts the changes from #363 that installed Node.js (via NodeSource) as part of the jenkins_data_engineering_new role, returning the Jenkins data engineering host provisioning to its prior behavior.

Changes:

  • Removed NodeSource GPG key installation, apt repository configuration, and Node.js package installation steps from the role tasks.
  • Removed the JENKINS_DATA_ENGINEERING_NODE_VERSION default variable from the role defaults.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
playbooks/roles/jenkins_data_engineering_new/tasks/main.yml Removes the NodeSource/Node.js installation tasks that were added in the reverted PR.
playbooks/roles/jenkins_data_engineering_new/defaults/main.yml Removes the default Node.js version variable introduced for the NodeSource repo setup.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@amits-2u
amits-2u merged commit 74bf99b into master Aug 4, 2026
2 of 4 checks passed
@amits-2u
amits-2u deleted the revert-363-rahul/install-node-for-de-jenkins-health-check branch August 4, 2026 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants