Skip to content

fix(SAS): license#760

Open
EveningStarlight wants to merge 1 commit intomasterfrom
edit-sas-license
Open

fix(SAS): license#760
EveningStarlight wants to merge 1 commit intomasterfrom
edit-sas-license

Conversation

@EveningStarlight
Copy link
Copy Markdown
Contributor

looks to fix the SAS license error.
image

Mirrors the fix used for The Zone:
StatCan/zone-kubeflow-containers#139

@EveningStarlight
Copy link
Copy Markdown
Contributor Author

Jose Matsuda

hey Evening in our tech elab today Souheil just showed me a message y'all got about SAS expiring, this is something we did and can see the pr here (its a bit different since we needed to do some shenaniganry before but this is what it essentially looks like now)
chore: change the SAS install to use image over storage account by Jose-Matsuda · Pull Request #139…
chore: change the SAS install to use image over storage account by Jose-Matsuda · Pull Request #139 · StatCan/zone-kubeflow-containers
Description
For https://jirab.statcan.ca/browse/ZONE-182
Do not merge because testing the built image that we are COPY --from
Tests / Quality Checks
Beta Process
Should this branch target &qu...

I DOUBT there's any issue just using this, since the license is unlimited, but note that it would need to be refreshed again in end of October (and we are considering moving to SAS M8 over M7 too lol)

maybe you could yap with whoever your people are to make sure its fine, but again i strongly doubt there's anything wrong with using an unlimited license (unless its like because external users then

the way the license gets updated is annoying as in you need to pull it to your local machine and then run their CLI to update it.

so in reality the entire image is part of the license update

like youd need to remove whatever SAS stuff gets installed in the AAW image and then add the following lines;

COPY --chown=jovyan:sasstaff --from=k8scc01covidacr.azurecr.io/sasinstallation:v1 /usr/local/SASHome /usr/local/SASHome
RUN ln -s /usr/local/SASHome/SASFoundation/9.4/bin/sas_en /usr/local/bin/sas
RUN chown root /usr/local/SASHome/SASFoundation/9.4/utilities/bin/elssrv &&
chown root /usr/local/SASHome/SASFoundation/9.4/utilities/bin/sasauth &&
chown root /usr/local/SASHome/SASFoundation/9.4/utilities/bin/sasperm &&
chmod 4755 /usr/local/SASHome/SASFoundation/9.4/utilities/bin/elssrv &&
chmod 4755 /usr/local/SASHome/SASFoundation/9.4/utilities/bin/sasauth &&
chmod 4755 /usr/local/SASHome/SASFoundation/9.4/utilities/bin/sasperm

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.

1 participant