Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add semi-automated CI stuff #19

Closed
wants to merge 6 commits into from

Conversation

merlijn-sebrechts
Copy link
Member

  • Terraform
  • Scripts
  • Useful overview links

@jnsgruk
Copy link
Member

jnsgruk commented Dec 5, 2023

Before this merges, we need to decide what to do with the Terraform state. In my own repos like this, I store the state encrypted in a (free tier) Azure object storage bucket (see here: https://github.com/jnsgruk/infra/blob/a3d5302e9189f45b7abbbaa8667e74d7d2be8494/provider.tf#L9-L14)

You can then provide the secret in CI on merge like this: https://github.com/jnsgruk/infra/blob/a3d5302e9189f45b7abbbaa8667e74d7d2be8494/.github/workflows/deploy.yaml#L25

Without the state file, it'll be much harder for anyone to actually use this :)

@@ -0,0 +1,6 @@
#!/usr/bin/bash
repos=("discord" "ffmpeg-sdk" "ci-screenshots" ".github" "mattermost-desktop" "signal-desktop" "ci" "ghvmctl" "sublime-text" "webkitgtk-sdk" "jenkins" "eclipse" "sentry" "obs-studio" "tmnationsforever" "scummvm" "xonotic" "atom" "offlineimap" "sommelier-core" "android-studio" "ffmpeg" "opentyrian" "spelunky" "flightgear" "sdlpop" "mumble" "corsixth" "magic-wormhole" "alacritty" "gimp" "helm" "sublime-merge" "minetest" "opentoonz" "fork-and-rename-me" "axel" "dosbox-x" "ddgr" "term2048" "arduino" "photoscape" "sommelier" "gradle" "s4a" "cumulonimbus" "android-studio-canary" "cassandra" "cryptool" "thelounge" "irssi" "marktext" "fkill" "pyradio" "mdl" "mosaic" "pogo" "unifi" "nano" "opentoonz-morevna" "kompozer" "get-iplayer" "simplenote" "mutt" "inadyn" "duckmarines" "brackets" "wordpress-desktop" "mrrescue" "wire" "tcpie" "slack-term" "links" "wethr" "qtctl" "bridge-designer" "climate-trail" "hello-snap-travis-ci")
Copy link
Member

Choose a reason for hiding this comment

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

I'd prefer to see this array declared with one repo per line, only because when we commit changes the diffs will be much easier to grok :)

terraform/README.md Outdated Show resolved Hide resolved
@merlijn-sebrechts
Copy link
Member Author

merlijn-sebrechts commented Dec 5, 2023

@jnsgruk This might need some discussion. I added the PR mainly for documentation purposes. So snapcrafters can see what I'm using, and so we have something to refer other people to who are interested in automating this.

I don't want to work on how to actually make this usable by other people yet. Do you think it makes sense to commit this as "docs to give you a general idea of what Merlijn is doing to scale himself", or do you think I should put it in another repo or something else?

@jnsgruk
Copy link
Member

jnsgruk commented Dec 5, 2023

@merlijn-sebrechts adding it here as docs is a good start, but honestly there isn't much to do to make it automated. We can make that leap another time, but it might get harder to move the state to somewhere central.

@merlijn-sebrechts
Copy link
Member Author

Closing this as I'll upload it to a separate repo. If we ever want to automate this properly, it will have to have specific permissions separate from what this repo provides, so it's better to just immediately add it in its own repo.

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.

2 participants