Skip to content

Add index page to the results bucket#46

Merged
phargogh merged 17 commits into
mainfrom
feature/24
Jun 23, 2026
Merged

Add index page to the results bucket#46
phargogh merged 17 commits into
mainfrom
feature/24

Conversation

@emlys

@emlys emlys commented Jun 15, 2026

Copy link
Copy Markdown
Member

Fixes #24

  • Adds a HTML/Javascript directory listing page to the results bucket. This is copied with some minor changes from the releases.naturalcapitalproject.org bucket. Allows users to browse workspace contents and download individual files.
  • Adds a custom results page to the pygeoapi site. This now includes both the GCS url to download the workspace, as well as a link to the directory listing page of the results bucket.
  • This PR also adds some infrastructure needed for the symposium demo plugin: a bucket for uploading client datastacks, which is publicly viewable but only writable if authenticated with a specific service account.

reserved_ip_range = module.private_service_access.reserved_ip_range
zone = var.zone
size_gb = 2560
filestore_tier = "BASIC_HDD"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Downgrading from SSD to HDD to reduce costs.

node_count_dynamic_max = 20
project_id = var.project_id
region = var.region
startup_script = "/home/bin/startup_script.sh"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The startup script has been removed, and all the environment setup now happens in the ansible playbook.

@emlys
emlys requested a review from phargogh June 23, 2026 00:22
@emlys emlys self-assigned this Jun 23, 2026
@emlys
emlys marked this pull request as ready for review June 23, 2026 00:23
@emlys emlys changed the title Feature/24 Add index page to the results bucket Jun 23, 2026

@phargogh phargogh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @emlys ! I didn't look super carefully at the bucket listing js code since it's adapted from the releases bucket listing code, but everything looks like it makes sense to me.

@phargogh
phargogh merged commit 53742cf into main Jun 23, 2026
2 checks passed
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.

Give the workspaces bucket a HTML page

2 participants