Fix build on GitHub Actions

[juliohm]

Hi @LakshyaKhatri, hope you are doing well.

In an attempt to port the Travis build to GitHub Actions I realized too late that you had added custom instructions in Travis to authenticate remotely with CDSAPI:

before_install:
- openssl aes-256-cbc -K $encrypted_fbf94d2678c4_key -iv $encrypted_fbf94d2678c4_iv
  -in .cdsapirc.enc -out $HOME\/.cdsapirc -d

Could you please help adding these instructions to our new CI.yml workflow? I also accidentally deleted the .cdsapirc.enc file you added to the repository. Please feel free to revive it if necessary.

[LakshyaKhatri]

Hey! Hi @juliohm ! Sorry for a late response. Sure, I'll take a look at it on my end

[juliohm]

Thank you @LakshyaKhatri ! Appreciate it! Looking forward to the fix ❤️

[LakshyaKhatri]

Hi @juliohm, I will need a little help on this. I was thinking to change the auth process this way:

1. Add the auth instructions to github environment variables
2. Now, before running the test suite we can create a .cdsapirc file on the fly using the details in the environment variables (and delete it in a finalization process).

That way we will not be maintaining a .cdsapirc.enc file on our repository. 😸💡
Or we can do it the old way i.e. add an encrypted version of .cdsapirc in our repo and decrypt it in the CI.

What do you suggest?

[LakshyaKhatri]

Ah! nvm, I just learned we have to store env variable values in the CI file itself 😶‍🌫️

[juliohm]

Thank you for taking a look into it. Any method that works and is safe should be fine. Em dom., 31 de jul. de 2022 03:44, Lakshya Khatri ***@***.***> escreveu:

…

Ah! nvm, I just learned we have to store env variable values in the CI file itself 😶‍🌫️ — Reply to this email directly, view it on GitHub <#46 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAZQW3J3WXSPQGFK6SPCK6DVWYOFTANCNFSM54TPJOHQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[juliohm]

The build is failing on Windows now because GRIB.jl does not support the platform.

[juliohm]

Opened an issue there: weech/GRIB.jl#14

[LakshyaKhatri]

Wondering if I should propose a PR to GRIB.jl. I'll try my hands on it once 🤞🏽

[juliohm]

If you have the time and knowledge to fix it, please feel free to contribute a fix ❤️

[juliohm]

@LakshyaKhatri did you have a chance to take a look into this?

[LakshyaKhatri]

Hi @juliohm, 👋🏽

I was still waiting for a response on GRIB.jl from the maintainer. But I will take GRIB.jl issue and work on it (after the vacation season though 🙂)

[maxstb]

In order to fix this, we first need to update the secrets for Actions (CDSAPI_URL and CDSAPI_KEY)
@juliohm Are you able to do this?

[juliohm]

In order to fix this, we first need to update the secrets for Actions (CDSAPI_URL and CDSAPI_KEY)
@juliohm Are you able to do this?

Will take a look at it by the end of the day. Thanks for checking it.

[juliohm]

@maxstb I've updated the secrets for Actions. Could you please check what else is necessary for the CI build?

[juliohm]

Fixed on master