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

breaking-change: Manage authorities #34

Merged
merged 11 commits into from
Aug 5, 2022

Conversation

valentindeaconu
Copy link
Collaborator

@valentindeaconu valentindeaconu commented Aug 1, 2022

This PR introduces authorities' management flows (including the UI).

An authority represents a namespace. Under that particular namespace, all resources are served to Terraform using the authority keys, name, and policy.

From the web interface, an authenticated user will be able to:

  • Create a new authority;
  • Remove an existing authority with all its data (gpg keys, api keys, providers and modules);
  • Allocate an API key for an existing authority;
  • Remove an existing API key from an existing authority.

From the API, an authenticated user (using an API key) will be able to:

  • Upload a new provider under the API key's authority namespace;
  • Upload a new module under the API key's authority namespace.

Note: The API key should be generated for that particular namespace.

However, this PR also introduces several breaking changes:

  • Adds a new owner column to the authority table;
  • Removes owner_email and owner_name from authority_api_keys table;
  • Adds authority_id column to the providers table;
  • Removes namespace column from providers table;
  • Adds authority_id column to the modules table;
  • Removes namespace column from modules table;
  • Rename authority table to authorities;
  • API endpoints for POST and DELETE methods for both providers and modules no longer take the namespace as an input parameter.

It's recommended to fully reconstruct the database.

@valentindeaconu valentindeaconu force-pushed the breaking-change/manage-authorities-from-ui branch from 33be855 to 3a8b7ba Compare August 5, 2022 14:58
@valentindeaconu valentindeaconu merged commit 13b0ab5 into master Aug 5, 2022
@valentindeaconu valentindeaconu deleted the breaking-change/manage-authorities-from-ui branch August 5, 2022 14:59
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

1 participant