Skip to content

Trigger acceptance tests ai on new pushes to main#1971

Open
jordimassaguerpla wants to merge 1 commit intoSUSE:masterfrom
jordimassaguerpla:fix_ai_pipelines_3
Open

Trigger acceptance tests ai on new pushes to main#1971
jordimassaguerpla wants to merge 1 commit intoSUSE:masterfrom
jordimassaguerpla:fix_ai_pipelines_3

Conversation

@jordimassaguerpla
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Jordi Massaguer Pla <jmassaguerpla@suse.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an automated trigger to run the MLM AI acceptance-test Jenkins pipeline when SCM changes are detected (intended for new pushes to main).

Changes:

  • Add an SCM polling trigger (pollSCM) to the declarative pipeline to check for changes every 5 minutes.

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

Comment on lines +21 to +26
triggers {
// Poll the SCM every five minutes for changes.
// 'H' is used to spread out the load on Jenkins.
// This requires the Jenkins job to be configured to point to the mcp-server-uyuni git repository.
pollSCM('H/5 * * * *')
}
Copy link

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

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

pollSCM only polls the Jenkins job’s configured SCM (or checkout scm in multibranch). This Jenkinsfile explicitly checks out https://github.com/uyuni-project/mcp-server-uyuni.git via the git step, which is not the SCM being polled, so this may not actually trigger on new pushes to mcp-server-uyuni/main and may instead poll whatever repo the job is configured with (often the repo containing this Jenkinsfile). If the intent is “run on pushes to mcp-server-uyuni main”, consider moving/configuring the job as “Pipeline script from SCM” pointing at that repo (so polling/webhooks track the right branch), or switch to a time-based cron(...) trigger and handle change detection explicitly inside the pipeline.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@copilot apply changes based on this feedback

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