vikunja
/
frontend
Archived
6
39
Fork 49
Code Issues 1 Pull Requests 4 Projects Releases 26 Activity

feature/improve-drone-config #2637

Merged
konrad merged 5 commits from dpschen/frontend:feature/improve-drone-config into main 2022-11-08 14:56:46 +00:00
Conversation 6 Commits 5 Files Changed 1 +23 -25
1 changed files with 23 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit da1d5eaba1 - Show all commits

48
.drone.yml
View File

@ -22,7 +22,7 @@ steps:
# Disabled until we figure out why it is so slow
# - name: restore-cache
# image: meltwater/drone-cache:dev
# pull: true
# pull: always
# environment:
# AWS_ACCESS_KEY_ID:
# from_secret: cache_aws_access_key_id
@ -41,7 +41,7 @@ steps:
- name: dependencies
image: node:18-alpine
pull: true
pull: always
environment:
dpschen commented 2022-11-05 16:02:20 +00:00
Outdated
Review
Copy Link

I forgot (and couldn't find out) if always is the default value either way. If so we could also remove the pull property.

I forgot (and couldn't find out) if `always` is the default value either way. If so we could also remove the `pull` property.
👍 1
dpschen commented 2022-11-07 12:51:09 +00:00
Outdated
Review
Copy Link

Do you know where to find the default config of drone?

Do you know where to find the default config of drone?
konrad commented 2022-11-07 13:00:51 +00:00
Outdated
Review
Copy Link

There's a reference somewhere.

I think for this particular case this is the relevant part of the docs: https://docs.drone.io/pipeline/kubernetes/syntax/images/#pulling-images

There's a reference somewhere. I think for this particular case this is the relevant part of the docs: https://docs.drone.io/pipeline/kubernetes/syntax/images/#pulling-images
dpschen commented 2022-11-07 13:44:24 +00:00
Outdated
Review
Copy Link

Yes, sadly they don't mention the default setting.

Yes, sadly they don't mention the default setting.
konrad commented 2022-11-07 14:05:00 +00:00
Outdated
Review
Copy Link

But they say:

If the image is tagged with latest, Drone will always attempt to pull the latest version of the image. Configure the runner to only pull the image if not found in the local cache:

But they say: > If the image is tagged with latest, Drone will always attempt to pull the latest version of the image. Configure the runner to only pull the image if not found in the local cache:
dpschen commented 2022-11-07 14:22:51 +00:00
Outdated
Review
Copy Link

Ahh, and ours are tagged with latest, so it will pull the latest anyway?

Ahh, and ours are tagged with latest, so it will pull the latest anyway?
konrad commented 2022-11-07 14:27:24 +00:00
Outdated
Review
Copy Link

This particular one has a tag of 18-alpine so it won't pull it always.

This particular one has a tag of `18-alpine` so it won't pull it always.
PNPM_CACHE_FOLDER: .cache/pnpm
CYPRESS_CACHE_FOLDER: .cache/cypress
@ -53,7 +53,7 @@ steps:
- name: lint
image: node:18-alpine
pull: true
pull: always
environment:
PNPM_CACHE_FOLDER: .cache/pnpm
commands:
@ -64,7 +64,7 @@ steps:
- name: build-prod
image: node:18-alpine
pull: true
pull: always
environment:
PNPM_CACHE_FOLDER: .cache/pnpm
commands:
@ -75,7 +75,7 @@ steps:
- name: test-unit
image: node:18-alpine
pull: true
pull: always
commands:
- corepack enable && pnpm config set store-dir .cache/pnpm
- pnpm run test:unit
@ -85,7 +85,7 @@ steps:
- name: typecheck
failure: ignore
image: node:18-alpine
pull: true
pull: always
environment:
PNPM_CACHE_FOLDER: .cache/pnpm
commands:
@ -96,7 +96,7 @@ steps:
- name: test-frontend
image: cypress/browsers:node16.14.0-chrome99-ff97
pull: true
pull: always
environment:
CYPRESS_API_URL: http://api:3456/api/v1
CYPRESS_TEST_SECRET: averyLongSecretToSe33dtheDB
@ -116,7 +116,7 @@ steps:
# - name: rebuild-cache
# image: meltwater/drone-cache:dev
# pull: true
# pull: always
# environment:
# AWS_ACCESS_KEY_ID:
# from_secret: cache_aws_access_key_id
@ -136,7 +136,7 @@ steps:
- name: deploy-preview
image: node:18-alpine
pull: true
pull: always
environment:
NETLIFY_AUTH_TOKEN:
from_secret: netlify_auth_token
@ -179,7 +179,7 @@ steps:
# - name: restore-cache
# image: meltwater/drone-cache:dev
# pull: true
# pull: always
# environment:
# AWS_ACCESS_KEY_ID:
# from_secret: cache_aws_access_key_id
@ -197,8 +197,7 @@ steps:
- name: build
image: node:18-alpine
pull: true
group: build-static
dpschen marked this conversation as resolved Outdated
dpschen commented 2022-11-05 16:03:51 +00:00
Outdated
Review
Copy Link

A group property doesn't seem to exist in drone.

A `group` property doesn't seem to exist in drone.
konrad commented 2022-11-07 12:00:54 +00:00
Outdated
Review
Copy Link

That's probably a relic from Drone 1

That's probably a relic from Drone 1
👍 1
pull: always
environment:
PNPM_CACHE_FOLDER: .cache/pnpm
commands:
@ -214,7 +213,7 @@ steps:
- name: static
image: kolaente/zip
pull: true
pull: always
commands:
- cd dist
- zip -r ../vikunja-frontend-unstable.zip *
@ -223,7 +222,7 @@ steps:
- name: release
image: plugins/s3
pull: true
pull: always
settings:
bucket: vikunja-releases
access_key:
@ -256,7 +255,7 @@ steps:
# - name: restore-cache
# image: meltwater/drone-cache:dev
# pull: true
# pull: always
# environment:
# AWS_ACCESS_KEY_ID:
# from_secret: cache_aws_access_key_id
@ -274,8 +273,7 @@ steps:
- name: build
image: node:18-alpine
pull: true
group: build-static
pull: always
environment:
PNPM_CACHE_FOLDER: .cache/pnpm
commands:
@ -291,7 +289,7 @@ steps:
- name: static
image: kolaente/zip
pull: true
pull: always
commands:
- cd dist
- zip -r ../vikunja-frontend-${DRONE_TAG##v}.zip *
@ -300,7 +298,7 @@ steps:
- name: release
image: plugins/s3
pull: true
pull: always
settings:
bucket: vikunja-releases
access_key:
@ -361,7 +359,7 @@ trigger:
steps:
- name: docker-unstable
image: plugins/docker:linux-arm
pull: true
pull: always
settings:
username:
from_secret: docker_username
@ -380,7 +378,7 @@ steps:
- name: docker-version
image: plugins/docker:linux-arm
pull: true
pull: always
settings:
username:
from_secret: docker_username
@ -400,7 +398,7 @@ steps:
- name: docker-unstable-arm64
image: plugins/docker:linux-arm64
pull: true
pull: always
settings:
username:
from_secret: docker_username
@ -419,7 +417,7 @@ steps:
- name: docker-version-arm64
image: plugins/docker:linux-arm64
pull: true
pull: always
settings:
username:
from_secret: docker_username
@ -461,7 +459,7 @@ trigger:
steps:
- name: docker-unstable
image: plugins/docker:linux-amd64
pull: true
pull: always
settings:
username:
from_secret: docker_username
@ -478,7 +476,7 @@ steps:
- name: docker-version
image: plugins/docker:linux-amd64
pull: true
pull: always
settings:
username:
from_secret: docker_username