Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Regeration script #45

Merged
merged 3 commits into from
Apr 22, 2022
Merged

Regeration script #45

merged 3 commits into from
Apr 22, 2022

Conversation

Anakin100100
Copy link
Contributor

I created the script requested in #39

How to use this script in cicd:

  • first make sure that all the env variables required for accessing the production db are present in the application docker container (I'm assuming that this is how we are going to run this because there everything is already prepared and installed), that would be DATABASE_HOST, DATABASE_NAME, DATABASE_USER, DATABASE_PASSWORD
  • second make sure that the server is running in the production mode in the background using RAILS_ENV=production rails s -b 127.0.0.1
  • third start the script in the production mode using RAILS_ENV=production rails r regeneration_script.rb

@mlazowik @micorix Do you need to know something more before you take care of incorporating this script into our ci workflow.

@Anakin100100 Anakin100100 requested a review from a team as a code owner April 17, 2022 14:18
wojtodzio
wojtodzio previously approved these changes Apr 21, 2022
Copy link
Contributor

@wojtodzio wojtodzio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤞

@Anakin100100
Copy link
Contributor Author

@wojtodzio thanks for the review. I don't know if we want to merge it yet. This script is mainly for incorporating it into the cicd and that is going to take some time so leaving the script on the master branch can wait. @mlazowik could you take care of that? @micorix is pretty busy with the frontend stuff and your help here would be really appreciated.

@micorix
Copy link
Member

micorix commented Apr 21, 2022

IMO script would be useful also for regenerating the DB manually so if it's possible I'd merge it

@Anakin100100
Copy link
Contributor Author

Ok, but let's commit the instructions on how to run the script in it first

@micorix micorix merged commit 907f173 into master Apr 22, 2022
@Anakin100100 Anakin100100 deleted the regeration_script branch May 14, 2022 13:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants