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

[IMP] runbot: reinforce version #852

Open
wants to merge 1 commit into
base: 17.0
Choose a base branch
from
Open

Conversation

d-fence
Copy link
Contributor

@d-fence d-fence commented Feb 20, 2024

When updating a bundle and changing the is_base field to True, it creates a new version based on the bundle name. This can potentially breaks builds and moreover, it can breaks the whole runbot when a duplicate version name is created.

With this commit:

  • a constraint on the version name uniqueness is added
  • the is_base field is hidden in the interface when the bundle name does not match the regular expression defined in the settings
  • a version cannot be created by the compute version if the bundle name does not match the above mentioned regular expression

@C3POdoo C3POdoo requested a review from a team February 20, 2024 15:59
@@ -50,7 +50,8 @@
<field name="project_id"/>
<field name="sticky" readonly="0"/>
<field name="to_upgrade" readonly="0"/>
<field name="is_base"/>
<field name="match_base" invisible="1"/>
<field name="is_base" invisible="not match_base"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

I would put it visible but readonly in all cases
This is a useful visual information but it makes no sense to modify it most of the time.

When updating a bundle and changing the is_base field to True, it
creates a new version based on the bundle name. This can potentially
breaks builds and moreover, it can breaks the whole runbot when a
duplicate version name is created.

With this commit:
- a constraint on the version name uniqueness is added
- the is_base field is now readonky
- a version cannot be created by the compute version if the version name
  does not match the above mentioned regular expression
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.

None yet

2 participants