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.

Sign up for GitHub

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

Jump to bottom

fix gdrive ds-load json #154

Closed
wants to merge 48 commits into from
Closed

fix gdrive ds-load json #154

wants to merge 48 commits into from

Conversation

florindragos
Copy link
Contributor

No description provided.

gertd and others added 30 commits July 22, 2023 21:40
@gertd
upd GetObject
83563f4
@gertd
complete config
38b3985
@carabasdaniel
Do not add to dependency map same address (#109)
f0f0e50
@gertd
config.yaml add authorizer -> reader dependency
f8b70cb
@gertd
GetObject revert ObjectTypeSelector validation logic
a2b5821
@gertd
REST API RC1
fd33621
@gertd
upd assets (#112)
b2f9e24
@gertd
OpenAPI support (#113)
e16809c 
* add OpenAPI support

* upd deps
@gertd
upd whitespace validation
c8f89dd
@gertd
upd whitespace validation
1c36f33
@gertd
upd whitespace validation
d007778
@gertd
add model service
b10f051
@gertd
config json schema
3f208b4
@gertd
merge sidecar PR from main
7640ab0
@gertd
update deps
4ede169
@gertd
merge main
1f382f0
@gertd
savepooint 20230913
4f9ac98
@glennblock
gdrive manifest and ds-load (#139)
86b433c 
* Initial commit

* gdrive it is

* Fixing image paths, and updating emails
Add console experience for topaz
8ab1e97
Fix linting
c95260b
Add console cli cmd
1784418
@ronenh
Replace 0.0.0.0 with localhost
38d2496
@oanatmaria
Merge pull request #142 from aserto-dev/console-config
577173a 
Replace 0.0.0.0 with localhost
Remove Println
260bd74
Get the latest commit from v3 go-edge-ds branch
911fd73
@ronenh
Use latest v3 console and fix handler paths
9c6b7a5
@ronenh
Add model service configuration
ab671f3
@oanatmaria
Merge pull request #143 from aserto-dev/console-config
2e8fed5 
Console config
Only copy the files of the console build folder
4f853dd
Expose default directory gateway port
357db1b
ronenh and others added 17 commits October 6, 2023 17:16
@ronenh
Merge pull request #134 from aserto-dev/add_console_exp
0323493 
Add console experience for topaz
@ronenh
Topaz with conosole (console06)
869ee99
@ronenh
clean pkg/app/console before build
26ca959
@ronenh
Bump to latest self-hosted-console
7a6cc40
@ronenh
Merge pull request #148 from aserto-dev/clean-console
397c652 
Bump self-hosted-console
@gertd
fix migrate test
5b5dfd9 
* fix migrate test
* add service cleanup
@gertd
Revert "fix migrate test"
42bf9e7 
This reverts commit 5b5dfd9.
@gertd
V3 only (#149)
8f045a5 
* migrate all builtins to be v3 based, adds handling for v2 and v3 request payloads, v2 request return v2 results, help * is always v3 based
* migrate and reorganize tests, remove gomega/ginko dependency
* add service cleanup, the cleanup function returned by wire does not cleanup, leaving the database open
@ronenh
Remove 'system' object from citadel data (#150)
cc168a8
@gertd
add manifest files
b5e1015
@ogazitt
updated citadel data in v3 branch
a8a2b83
@ogazitt
Merge pull request #152 from aserto-dev/fix/citadel-data-v3
5c18154 
updated citadel data in v3 branch
@gertd
upd v2 builtins & getUserFromIdentity
f64d417
@gertd
upd topaz test command
b4e692c
@gertd
upd ci.yaml
8e7530d
@gertd
ci.yaml add CONSOLE_VERSION
179165a
@gertd
int automatic schema migration
b654b4c
@gertd
Copy link
Member

gertd commented Oct 27, 2023

Please note:

  1. The data format is a V2 format, which cannot be used as we need subject_relation to be populated, so this must be updated to a V3 format
  2. There has been no validation of the manifest or data so far, so assume this is NOT final

gertd
gertd requested changes Oct 27, 2023
View reviewed changes
Copy link
Member

@gertd gertd left a comment

Choose a reason for hiding this comment

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

The object and relation formats used are v2 formats, gDrive requires subject_relation to be able to be expressed and set in the relations; hence the data format should be updated to V3 object and relation shapes.

Base automatically changed from v3 to main November 4, 2023 02:50
@gertd
Copy link
Member

gertd commented Nov 16, 2023

@florindragos is this PR still needed?

@gertd gertd deleted the fix_gdrive branch November 28, 2023 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gertd gertd gertd requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@florindragos @gertd @carabasdaniel @glennblock @ronenh @oanatmaria @ogazitt