AssertionError: 400 != 302 in master CI #57

New Issue

Closed

opened 2022-09-11 20:21:30 +00:00 by dachary · 2 comments

dachary commented

2022-09-11 20:21:30 +00:00

Owner

There are errors in the master CI

https://woodpecker.gna.org/Hostea/dashboard/build/177/6

This happened right after merging https://gitea.gna.org/Hostea/dashboard/pulls/55

There are errors in the master CI https://woodpecker.gna.org/Hostea/dashboard/build/177/6 This happened right after merging https://gitea.gna.org/Hostea/dashboard/pulls/55

👀 1

dachary commented

2022-09-11 20:22:08 +00:00

Poster

Owner

@realaravinth it does not look related to the name restriction change. Any idea?

dachary referenced this issue from Gna/organization

2022-09-11 20:24:36 +00:00

[hosteadashboard] deployment, diagnostic and bug reports #12

realaravinth commented

2022-09-11 20:46:57 +00:00

Owner

Good catch!

It seems to be related. That PR restricts instance names to alphanumeric characters and requires them to be less than 20 characters. Some of the instance names used in the tests vioalte this requirement.

I don't understand how slipped manual testing on my side and how Woodpecker didn't get triggered when the PR was opened.

Working on a fix

Good catch! It seems to be related. That PR restricts instance names to alphanumeric characters and requires them to be less than 20 characters. Some of the instance names used in the tests vioalte this requirement. I don't understand how slipped manual testing on my side and how Woodpecker didn't get triggered when the PR was opened. Working on a fix

👍 1