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

chore: Remove search index publish scripts #536

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

chore: Remove search index publish scripts #536

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 0 additions & 4 deletions Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,16 +16,12 @@ ENV GO111MODULE=on
USER root
RUN go get -d -v
RUN make validate-schema
RUN make publish-search-index-dry-run
RUN make parse-services
RUN make generate-search-index
RUN CGO_ENABLED=1 go build -o /go/bin/chrome-service-backend
# Build the migration binary.
RUN CGO_ENABLED=1 go build -o /go/bin/chrome-migrate cmd/migrate/migrate.go

# Build the search index binary.
RUN CGO_ENABLED=1 go build -o /go/bin/chrome-search-index cmd/search/publishSearchIndex.go

FROM registry.redhat.io/ubi8-minimal:latest

# Setup permissions to allow RDSCA to be written from clowder to container
Expand Down
10 changes: 1 addition & 9 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,15 +38,7 @@ clean:

validate-schema:
go run cmd/validate/*

publish-search-index:
go run cmd/search/*

publish-search-index-dry-run: export SEARCH_INDEX_DRY_RUN = true

publish-search-index-dry-run:
go run cmd/search/*


generate-search-index: export SEARCH_INDEX_WRITE = true

generate-search-index:
Expand Down
0 cmd/search/publishSearchIndex.go → cmd/search/generateSearchIndex.go
View file
Open in desktop
File renamed without changes.
6 changes: 0 additions & 6 deletions deploy/clowdapp.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,12 +36,6 @@ objects:
- -c
- chrome-migrate
inheritEnv: true
- name: publish-search-index
command:
- bash
- -c
- chrome-search-index
inheritEnv: true
livenessProbe:
failureThreshold: 3
httpGet:
Expand Down
8 changes: 0 additions & 8 deletions docs/search-index.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,11 +75,3 @@ To generate the search index follow these steps:

1. Make sure the correct files were changed. If you are changing navigation files, make sure to edit both stage and production files.
2. Run the `make generate-search-index` command from the project root

## Publishing search index

To publish the search index follow these steps:

1. Make sure the correct files were changed. If you are changing navigation files, make sure to edit both stage and production files.
2. Make sure you are on RH wired network or on VPN
3. Run the `make publish-search-index` command from the project root
Loading