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

Issue with EULA #46623

Closed
thomasonsignup opened this issue Jan 22, 2020 — with docs.microsoft.com · 5 comments
Closed

Issue with EULA #46623

thomasonsignup opened this issue Jan 22, 2020 — with docs.microsoft.com · 5 comments

Comments

Copy link

Getting error:
Missing EULA=accept command line option. You must provide this to continue. after deploying the Container Instance.

EULA is configured in environmentVariables as "name": "EULA", "value": "accept"


Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

@GiftA-MSFT
Copy link
Contributor

@thomasonsignup thanks for your feedback. We will look into this issue and get back to you shortly.

@IEvangelist
Copy link
Member

Hi @thomasonsignup,

Could you please provide the YAML file that you used from the instructions? Be mindful to remove API keys and other personally identifiable / sensitive data values.

If you follow the YAML in the example, you'll see that we explicitly specify eula: accept in environmentVariables:

apiVersion: 2018-10-01
location: # < Valid location >
name: # < Container Group name >
imageRegistryCredentials: # This is required when pulling a non-public image
  - server: containerpreview.azurecr.io
    username: # < The username for the preview container registry >
    password: # < The password for the preview container registry >
properties:
  containers:
  - name: # < Container name >
    properties:
      image: # < Repository/Image name >
      environmentVariables: # These env vars are required
        - name: eula
          value: accept
        - name: billing
          value: # < Service specific Endpoint URL >
        - name: apikey
          value: # < Service specific API key >
      resources:
        requests:
          cpu: 4 # Always refer to recommended minimal resources
          memoryInGb: 8 # Always refer to recommended minimal resources
      ports:
        - port: 5000
  osType: Linux
  volumes: # This node, is only required for container instances that pull their model in at runtime, such as LUIS.
  - name: aci-file-share
    azureFile:
      shareName: # < File share name >
      storageAccountName: # < Storage account name>
      storageAccountKey: # < Storage account key >
  restartPolicy: OnFailure
  ipAddress:
    type: Public
    ports:
    - protocol: tcp
      port: 5000
tags: null
type: Microsoft.ContainerInstance/containerGroups

Copy link
Author

Hello, I've tried lowercase as well, same issue.
The YAML does not work and throws an error that we do not have access to the repository.
"The image 'containerpreview.azurecr.io/microsoft/cognitive-services-form-recognizer-custom-supervised-labeltool:latest' in container group 'sample-labeling-tool-container' is not accessible. Please check the image and registry credential."

I ended up doing it with ARM Templates with the details from the prepared yaml:
apiVersion: 2018-10-01
location: West Europe
name: sample-labeling-tool-container
imageRegistryCredentials: # This is required when pulling a non-public image

  • server: containerpreview.azurecr.io
    username: xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxx
    password: xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxx
    properties:
    containers:
    • name: sample-labeling-tool-container
      properties:
      image: containerpreview.azurecr.io/microsoft/cognitive-services-form-recognizer-custom-supervised-labeltool:latest
      environmentVariables: # These env vars are required
      - name: eula
      value: accept
      - name: billing
      value: https://ml1-form-rec.cognitiveservices.azure.com/
      - name: apikey
      value: xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
      resources:
      requests:
      cpu: 4 # Always refer to recommended minimal resources
      memoryInGb: 8 # Always refer to recommended minimal resources
      ports:
      - port: 5000
      osType: Linux

volumes: # This node, is only required for container instances that pull their model in at runtime, such as LUIS.

- name: aci-file-share

azureFile:

shareName: # < File share name >

storageAccountName: # < Storage account name>

storageAccountKey: # < Storage account key >

restartPolicy: OnFailure
ipAddress:
type: Public
ports:
- protocol: tcp
port: 5000
tags: null
type: Microsoft.ContainerInstance/containerGroups

You can see the env variables from the generated template:
{
"$schema": "https://schema.management.azure.com/schemas/2015-01-01/deploymentTemplate.json#",
"contentVersion": "1.0.0.0",
"parameters": {
"containerGroups_sample_labeling_tool_container_name": {
"defaultValue": "sample-labeling-tool-container",
"type": "String"
}
},
"variables": {},
"resources": [
{
"type": "Microsoft.ContainerInstance/containerGroups",
"apiVersion": "2018-10-01",
"name": "[parameters('containerGroups_sample_labeling_tool_container_name')]",
"location": "westeurope",
"properties": {
"containers": [
{
"name": "[parameters('containerGroups_sample_labeling_tool_container_name')]",
"properties": {
"image": "containerpreview.azurecr.io/microsoft/cognitive-services-form-recognizer-custom-supervised-labeltool:latest",
"ports": [
{
"protocol": "TCP",
"port": 5000
}
],
"environmentVariables": [
{
"name": "eula",
"value": "accept"
},
{
"name": "billing",
"value": "https://ml1-form-rec.cognitiveservices.azure.com/"
},
{
"name": "apikey",
"value": "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
}
],
"resources": {
"requests": {
"memoryInGB": 8,
"cpu": 4
}
}
}
}
],
"imageRegistryCredentials": [
{
"server": "containerpreview.azurecr.io",
"username": "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxx",
"password": "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxx"
}
],
"restartPolicy": "OnFailure",
"ipAddress": {
"ports": [
{
"protocol": "TCP",
"port": 5000
}
],
"type": "Public"
},
"osType": "Linux"
}
}
]
}

Copy link
Author

For future reference this needs to be added to commands to make it work (ARM template below)
"command": [
"./run.sh", "eula=accept"
],

@IEvangelist
Copy link
Member

You must first request access to the container registry. #please-close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants