-
Notifications
You must be signed in to change notification settings - Fork 106
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
Regex for numbers only work with numbers up to 0999 #129
Comments
I also tried other regex which are more specific which did have the same behavior:
This one works, but it is kind of dirty:
|
I also tested this with concourse v5.3.0 and same behaviour. |
Weird. Can you define 'works'? What exactly does/doesn't it do? Is the version never detected? Is it detected, but in an incorrect order relative to the other versions? (You could check with |
we just ran into the same issue. resources:
- name: Artifact
type: s3
bucket: super-duper-artifact-bucket
regexp: root/foo/bar_(.*).zip
# regexp: root/foo/bar_(1[0-9]+)\.zip
access_key_id: ((access_key_id))
secret_access_key: ((secret_access_key))
{...}
jobs:
- name: Build Docker Image
plan:
- get: Artifact
{...} We did the qick and dirty "fix" in the regex to fly --target foo resource-versions --resource my_pipeline/Artifact
id version enabled
5938003 path:root/foo/bar_1006.zip yes
5937128 path:root/foo/bar_1005.zip yes
5935573 path:root/foo/bar_1004.zip yes
5935390 path:root/foo/bar_1002.zip yes We are using Concourse version: v5.8.0 |
We have a S3 Bucket where we store files with this pattern:
The pipline is using this resource:
And also gets triggered:
This is has worked until the number changed:
folder1/folder2/static_string_0999.zip <-- Works
folder1/folder2/static_string_1000.zip <-- Does not work
For me this seems to be a bug.
If important:
We are using concourse Version: v5.0.1
Can you confirm?
Will this be fixed?
Is there a good workaround in place for the time?
The text was updated successfully, but these errors were encountered: