Skip to content

Commit

Permalink
chore: remove hack for materializing keys (#2211)
Browse files Browse the repository at this point in the history
  • Loading branch information
mhamilton723 authored Apr 18, 2024
1 parent 786e59a commit 1ed7dd2
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 15 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -132,8 +132,6 @@ class SearchWriterSuite extends TestBase with AzureSearchKey with IndexJsonGette

override def beforeAll(): Unit = {
println("WARNING CREATING SEARCH ENGINE!")
println("materializing keys for tests")
println((openAIAPIKey + openAIAPIKeyGpt4 + openAIServiceName + cognitiveKey).length)
SearchIndex.createIfNoneExists(azureSearchKey,
testServiceName,
createSimpleIndexJson(indexName))
Expand Down
13 changes: 1 addition & 12 deletions pipeline.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ jobs:
artifactsFeeds: SynapseML_PublicPackages
mavenServiceConnections: SynapseML_PublicPackages-Feed-Connection
- task: AzureCLI@2
displayName: 'Publish Artifacts 1'
displayName: 'Publish Artifacts'
inputs:
azureSubscription: 'SynapseML Build'
scriptLocation: inlineScript
Expand All @@ -104,17 +104,6 @@ jobs:
sudo apt-get install graphviz doxygen -y
source activate synapseml
sbt packagePython uploadNotebooks
env:
SYNAPSEML_ENABLE_PUBLISH: true
- task: AzureCLI@2
displayName: 'Publish Artifacts 2'
inputs:
azureSubscription: 'SynapseML Build'
scriptLocation: inlineScript
scriptType: bash
inlineScript: |
set -e
source activate synapseml
sbt -DskipCodegen=true publishBlob publishDocs publishR publishPython
sbt genBuildInfo
echo "##vso[task.uploadsummary]$(pwd)/target/Build.md"
Expand Down
2 changes: 1 addition & 1 deletion templates/update_cli.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@ steps:
versionSpec: '8'
jdkArchitectureOption: 'x64'
jdkSourceOption: 'PreInstalled'
- bash: pip install azure-cli==2.59.0
- bash: pip install azure-cli==2.58.0
displayName: 'Upgrade Azure CLI'

0 comments on commit 1ed7dd2

Please sign in to comment.