Skip to content

codeintel: add an --associated-index-id flag for autoindexer usage#430

Merged
garobrik merged 4 commits into
mainfrom
garo/lsif_upload_associatedIndexId
Jan 18, 2021
Merged

codeintel: add an --associated-index-id flag for autoindexer usage#430
garobrik merged 4 commits into
mainfrom
garo/lsif_upload_associatedIndexId

Conversation

@garobrik

Copy link
Copy Markdown
Contributor

Need to wait for the codeintelutils update before this works.

Comment thread go.mod Outdated
Comment thread cmd/src/lsif_upload.go Outdated
@@ -71,6 +72,7 @@ Examples:
flags.ignoreUploadFailures = flagSet.Bool("ignore-upload-failure", false, `Exit with status code zero on upload failure.`)
flags.uploadRoute = flagSet.String("upload-route", "/.api/lsif/upload", "The path of the upload route.")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we add 'internal use only' here too?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure, if someone's changed this in a forked Sourcegraph instance they'll happily ignore that instruction.

Comment thread cmd/src/lsif_upload.go
flags.maxPayloadSizeMb = flagSet.Int("max-payload-size", 100, `The maximum upload size (in megabytes). Indexes exceeding this limit will be uploaded over multiple HTTP requests.`)
flags.ignoreUploadFailures = flagSet.Bool("ignore-upload-failure", false, `Exit with status code zero on upload failure.`)
flags.uploadRoute = flagSet.String("upload-route", "/.api/lsif/upload", "The path of the upload route.")
flags.uploadRoute = flagSet.String("upload-route", "/.api/lsif/upload", "The path of the upload route. For internal use only.")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

image

@efritz efritz self-requested a review January 18, 2021 23:15
@garobrik garobrik merged commit b7ed8a5 into main Jan 18, 2021
@garobrik garobrik deleted the garo/lsif_upload_associatedIndexId branch January 18, 2021 23:17
scjohns pushed a commit that referenced this pull request Apr 24, 2023
)

* accept and pass associated-index-id flag

* bump codeintelutils version

* bump codeintelutils version again

* mark 'upload-route' flag "for internal use only"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants