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

Add SQL Server 2019 workflow contained on runner VM #110

Merged
merged 98 commits into from
Jun 27, 2024

Conversation

aaronburtle
Copy link
Contributor

@aaronburtle aaronburtle commented Jun 17, 2024

This PR adds a new .yaml file which will handle the transition of SQL2019 ML Utils from utilizing Azure Resources. This aligns SQLMLUtils for SQL Server 2019 with the first phase of the security push.

Errors that are happening during the running of the pipeline are carried over from the previous SQLMLUtils environment and are related to version updates that need be properly handled. These changes are out of scope for this PR.

To demonstrate that the SQL Server 2019 environment is setup correctly we have added a SPEES task both for Python and R, so that we can validate that the GHA is capable of correctly executing an external script using the machine learning services.

Successful pipeline run: https://github.com/microsoft/sqlmlutils/actions/runs/9703488737

.github/workflows/SQL2019.yaml Outdated Show resolved Hide resolved
.github/workflows/SQL2019.yaml Show resolved Hide resolved
.github/workflows/SQL2019.yaml Outdated Show resolved Hide resolved
.github/workflows/SQL2019.yaml Outdated Show resolved Hide resolved
.github/workflows/SQL2019.yaml Show resolved Hide resolved
.github/workflows/SQL2019.yaml Show resolved Hide resolved
.github/workflows/SQL2019.yaml Show resolved Hide resolved
.github/workflows/ci.yaml Outdated Show resolved Hide resolved
Copy link
Contributor

@seantleonard seantleonard left a comment

Choose a reason for hiding this comment

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

pending 2 nits

.github/workflows/SQL2019.yaml Show resolved Hide resolved
.github/workflows/SQL2019.yaml Show resolved Hide resolved
@seantleonard seantleonard merged commit 8c39055 into master Jun 27, 2024
2 of 6 checks passed
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.

None yet

2 participants