rename nwp service to include metoffice
peterdudfield opened this issue · 7 comments
Rename nwp service. To be nwp to nwp-metoffice https://github.com/openclimatefix/ocf-infrastructure/blob/main/terraform/nowcasting/development/main.tf#L138
Similarily replace data-national with data-metoffice
Would be good to do this on development first then we can move over to production
Hello @peterdudfield, I would like to take this issue up
If I understand it correctly, I just have to rename the nwp to nw-metoffice, so will I have to change the ecs-task_name name variable right? Is there something else that will have to be done
Also, can you give me some more insights on how to setup ocf-infrastructure to test this out, after following this setup guide, will I have to run some tests?
Hi @aatmanvaidya thanks so much for taking this up.
Yea thats right, nwp to nwp-metoffice.
testing is now done through CI. So unless you know terraform very well, it might be good to just let the CI do its thing. Sorry if that is very helpful. I would b interested to know if we can test it a different way. terraform validate will get you so far
@peterdudfield understood, will work on this and make a PR at the earliest!
Thank you
@peterdudfield I have some small doubts regarding setting up terraform
When I did terraform init inside nowcasting/development, I got an error saying I will have to do terraform login first, I created a login token, but now where do I store/enter this login token? Will I have to create a config file?
Apologies for a silly question, I am a beginner to terraform.
yea, sorry, Im not quite sure. You might have to go and read about terraform set up. Or it might be becasue we are setup with the terraform-cloud
@peterdudfield I have raised a PR - #476
please let me know if any changes are required,
thank you for your support all along.