Zenika-Training / strigo-init-script-libs

Script snippets for Strigo init scripts
3 stars 1 forks source link

Add Windows init scripts #23

Closed zigarn closed 3 years ago

CedricRup commented 3 years ago

J'essaie par mail pour voir : la c'est mineur et ça fait apparaître le pourquoi à travers la constante... À toi de voir

Le dim. 21 nov. 2021 à 15:38, Alexandre Garnier @.***> a écrit :

@.**** commented on this pull request.

In win_scripts/chocolatey.ps1 https://github.com/Zenika/strigo-init-script-libs/pull/23#discussion_r753809285 :

@@ -0,0 +1,2 @@

+# Install Chocolatey (https://community.chocolatey.org/courses/installation/installing?method=install-from-powershell-v3)

+Set-ExecutionPolicy Bypass -Scope Process -Force; [System.Net.ServicePointManager]::SecurityProtocol = [System.Net.ServicePointManager]::SecurityProtocol -bor 3072; iex ((New-Object System.Net.WebClient).DownloadString('https://community.chocolatey.org/install.ps1'))

À ce niveau là, ce n'est qu'un copier/coller de la commande fournie par la doc de chocolatey. Donc je ne sais pas dans quelle mesure il est préférable de la conserver tel quel ou de l'améliorer.

— You are receiving this because your review was requested. Reply to this email directly, view it on GitHub https://github.com/Zenika/strigo-init-script-libs/pull/23#discussion_r753809285, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMZUTQTMKJS7CEC3VS3IALUND74VANCNFSM5IOE2IWA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

CedricRup commented 3 years ago

Ah, peut être pas en effet.. Et j'ai pas l'équivalent sous la main, donc je retire mon commentaire

Le dim. 21 nov. 2021 à 15:41, Alexandre Garnier @.***> a écrit :

@.**** commented on this pull request.

In win_scripts/strigo.ps1 https://github.com/Zenika/strigo-init-script-libs/pull/23#discussion_r753809830 :

@@ -0,0 +1,10 @@

+# Install https://www.powershellgallery.com/packages/AWS.Tools.EC2 to have 'Get-EC2InstanceMetadata'

+Install-Module -Name AWS.Tools.EC2

+

+# Inject Strigo context

+[System.Environment]::SetEnvironmentVariable("INSTANCE_NAME", "{{ .STRIGO_RESOURCE_NAME }}", [System.EnvironmentVariableTarget]::Machine)

Ben j'ai trouvé plusieurs façons de faire en cherchant, celle-ci me semblait la plus complète/explicite.

@CedricRup https://github.com/CedricRup La forme que tu proposes va bien persister la variable d'environnement au niveau système (et donc tout process futur en héritera) ? Pas juste au niveau du script lui-même ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Zenika/strigo-init-script-libs/pull/23#discussion_r753809830, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMZUTW7OBIKW5ZJ33HCLJ3UNEALPANCNFSM5IOE2IWA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.