Re: [PR] Changed SDAP startup behavior to wait for all datasets to be ready [sdap-nexus]

2024-06-07 Thread via GitHub
RKuttruff merged PR #314: URL: https://github.com/apache/sdap-nexus/pull/314 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[PR] Changed SDAP startup behavior to wait for all datasets to be ready [sdap-nexus]

2024-06-07 Thread via GitHub
RKuttruff opened a new pull request, #314: URL: https://github.com/apache/sdap-nexus/pull/314 Originally part of #307 moved to its own PR for the upcoming 1.3.0 release cycle -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] data-access patch [sdap-nexus]

2024-06-06 Thread via GitHub
RKuttruff merged PR #313: URL: https://github.com/apache/sdap-nexus/pull/313 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-518 - Collection Config Docs [sdap-nexus]

2024-06-06 Thread via GitHub
RKuttruff merged PR #311: URL: https://github.com/apache/sdap-nexus/pull/311 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-518 - Collection Config Docs [sdap-nexus]

2024-06-06 Thread via GitHub
kevinmarlis commented on code in PR #311: URL: https://github.com/apache/sdap-nexus/pull/311#discussion_r1625117851 ## docs/collections.rst: ## Review Comment: Maybe add something about the commonly used slice dimensions (30x30 or 60x60) for the protobuf grid section.

Re: [PR] SDAP-518 - Collection Config Docs [sdap-nexus]

2024-06-06 Thread via GitHub
kevinmarlis commented on code in PR #311: URL: https://github.com/apache/sdap-nexus/pull/311#discussion_r1625117851 ## docs/collections.rst: ## Review Comment: Maybe add something about the commonly used slice dimensions (30x30 or 60x60) for the protobuf grid section.

Re: [PR] data-access patch [sdap-nexus]

2024-06-06 Thread via GitHub
RKuttruff commented on code in PR #313: URL: https://github.com/apache/sdap-nexus/pull/313#discussion_r1630362975 ## data-access/nexustiles/backends/zarr/backend.py: ## @@ -42,7 +42,8 @@ class ZarrBackend(AbstractTileService): def __init__(self, dataset_name, path,

Re: [PR] data-access patch [sdap-nexus]

2024-06-06 Thread via GitHub
wphyojpl commented on code in PR #313: URL: https://github.com/apache/sdap-nexus/pull/313#discussion_r1630358090 ## data-access/nexustiles/backends/zarr/backend.py: ## @@ -42,7 +42,8 @@ class ZarrBackend(AbstractTileService): def __init__(self, dataset_name, path,

Re: [PR] SDAP-517 - Credential management for AWS collections [sdap-nexus]

2024-06-06 Thread via GitHub
RKuttruff commented on PR #312: URL: https://github.com/apache/sdap-nexus/pull/312#issuecomment-2152980912 Some changes of this PR were copied to #313 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[PR] Backends patch [sdap-nexus]

2024-06-06 Thread via GitHub
RKuttruff opened a new pull request, #313: URL: https://github.com/apache/sdap-nexus/pull/313 - Code cleanup - Added dask as dependency, so it will be utilized with Zarr datasets - Fixed creation of tile times array from Zarr data to ensure its in seconds from 1970-01-01. Original

[PR] SDAP-517 - Credential management for AWS collections [sdap-nexus]

2024-06-05 Thread via GitHub
RKuttruff opened a new pull request, #312: URL: https://github.com/apache/sdap-nexus/pull/312 This PR augments the configurations of Zarr (and eventually future) collections. Initially, collections in non-public S3 buckets needed to be provided an Access Key ID and Secret Access Key

[PR] SDAP-518 - Collection Config Docs [sdap-nexus]

2024-06-03 Thread via GitHub
RKuttruff opened a new pull request, #311: URL: https://github.com/apache/sdap-nexus/pull/311 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

Re: [PR] Updates to openapi [sdap-nexus]

2024-05-20 Thread via GitHub
kevinmarlis merged PR #310: URL: https://github.com/apache/sdap-nexus/pull/310 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-490 - expanding openapi spec [sdap-nexus]

2024-05-20 Thread via GitHub
kevinmarlis commented on PR #281: URL: https://github.com/apache/sdap-nexus/pull/281#issuecomment-2121031185 Closing in favor of #310 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] SDAP-490 - expanding openapi spec [sdap-nexus]

2024-05-20 Thread via GitHub
kevinmarlis closed pull request #281: SDAP-490 - expanding openapi spec URL: https://github.com/apache/sdap-nexus/pull/281 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[PR] Updates to openapi [sdap-nexus]

2024-05-20 Thread via GitHub
kevinmarlis opened a new pull request, #310: URL: https://github.com/apache/sdap-nexus/pull/310 Added additional analysis algorithms: - timeAvgMapSpark - timeSeriesSpark - dailydifferenceaverage_spark - latitudeTimeHofMoellerSpark - longitudeTimeHofMoellerSpark -

Re: [PR] Attempted fix for failing RTD build [sdap-nexus]

2024-05-20 Thread via GitHub
RKuttruff merged PR #309: URL: https://github.com/apache/sdap-nexus/pull/309 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Attempted fix for failing RTD build [sdap-nexus]

2024-05-20 Thread via GitHub
RKuttruff commented on PR #309: URL: https://github.com/apache/sdap-nexus/pull/309#issuecomment-2120980010 Build passes https://readthedocs.org/projects/incubator-sdap-nexus/builds/24434034/ -- This is an automated message from the Apache Git Service. To respond to the message, please

[PR] Attempted fix for failing RTD build [sdap-nexus]

2024-05-20 Thread via GitHub
RKuttruff opened a new pull request, #309: URL: https://github.com/apache/sdap-nexus/pull/309 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

Re: [PR] Fixed install of kubectl in Collection Manager image build [sdap-ingester]

2024-05-13 Thread via GitHub
RKuttruff merged PR #99: URL: https://github.com/apache/sdap-ingester/pull/99 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-502: Gridded tile generation bug patch [sdap-ingester]

2024-05-09 Thread via GitHub
RKuttruff merged PR #92: URL: https://github.com/apache/sdap-ingester/pull/92 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-472 - Support for defining gridded Zarr datasets from the collection config [sdap-ingester]

2024-05-09 Thread via GitHub
RKuttruff merged PR #86: URL: https://github.com/apache/sdap-ingester/pull/86 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] CM improvements [sdap-ingester]

2024-05-09 Thread via GitHub
RKuttruff merged PR #93: URL: https://github.com/apache/sdap-ingester/pull/93 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-512 - Have GI close ZK/Solr/Cassandra connections when done with writing to Solr [sdap-ingester]

2024-05-09 Thread via GitHub
RKuttruff merged PR #97: URL: https://github.com/apache/sdap-ingester/pull/97 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Added some missing ASF headers and removed some unneeded files [sdap-nexus]

2024-05-08 Thread via GitHub
RKuttruff merged PR #302: URL: https://github.com/apache/sdap-nexus/pull/302 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-513 - Configurable Solr init pod image in helm [sdap-nexus]

2024-05-08 Thread via GitHub
RKuttruff merged PR #306: URL: https://github.com/apache/sdap-nexus/pull/306 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP 515 - Improved handling of unreachable remote SDAPs [sdap-nexus]

2024-05-02 Thread via GitHub
kevinmarlis merged PR #308: URL: https://github.com/apache/sdap-nexus/pull/308 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[PR] SDAP 515 - Improved handling of unreachable remote SDAPs [sdap-nexus]

2024-05-02 Thread via GitHub
kevinmarlis opened a new pull request, #308: URL: https://github.com/apache/sdap-nexus/pull/308 This improves how unreachable remote SDAPs are handled during calls to `/nexus/list`. Previously, only a few reasons for being unable to connect to the remote SDAP (like timeouts) were being

[PR] SDAP-484 - Cloud-optimized GeoTIFF support [incubator-sdap-nexus]

2024-04-04 Thread via GitHub
RKuttruff opened a new pull request, #307: URL: https://github.com/apache/incubator-sdap-nexus/pull/307 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[PR] Fixed install of kubectl in Collection Manager image build [incubator-sdap-ingester]

2024-03-27 Thread via GitHub
RKuttruff opened a new pull request, #99: URL: https://github.com/apache/incubator-sdap-ingester/pull/99 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] Update quickstart with new solr command [incubator-sdap-nexus]

2024-03-27 Thread via GitHub
RKuttruff commented on PR #298: URL: https://github.com/apache/incubator-sdap-nexus/pull/298#issuecomment-2023432834 Thinking on this, it may be a good idea to provide both commands and inform the user that if one does not work, try the other. -- This is an automated message from the

[PR] SDAP-513 - Configurable Solr init pod image in helm [incubator-sdap-nexus]

2024-03-27 Thread via GitHub
RKuttruff opened a new pull request, #306: URL: https://github.com/apache/incubator-sdap-nexus/pull/306 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] Forward 1.2.0 release to develop [incubator-sdap-nexus]

2024-03-21 Thread via GitHub
RKuttruff closed pull request #303: Forward 1.2.0 release to develop URL: https://github.com/apache/incubator-sdap-nexus/pull/303 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] SDAP-469 - Extra modifications for elevation support [incubator-sdap-nexusproto]

2024-03-20 Thread via GitHub
RKuttruff merged PR #19: URL: https://github.com/apache/incubator-sdap-nexusproto/pull/19 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Sync develop to master [incubator-sdap-ingester]

2024-03-20 Thread via GitHub
RKuttruff merged PR #98: URL: https://github.com/apache/incubator-sdap-ingester/pull/98 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Merge master into develop post-1.2.0 release [incubator-sdap-nexus]

2024-03-20 Thread via GitHub
RKuttruff merged PR #305: URL: https://github.com/apache/incubator-sdap-nexus/pull/305 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[PR] Merge master into develop post-1.2.0 release [incubator-sdap-nexus]

2024-03-20 Thread via GitHub
RKuttruff opened a new pull request, #305: URL: https://github.com/apache/incubator-sdap-nexus/pull/305 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] 1.2.0 release to develop [incubator-sdap-nexus]

2024-03-18 Thread via GitHub
RKuttruff closed pull request #304: 1.2.0 release to develop URL: https://github.com/apache/incubator-sdap-nexus/pull/304 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[PR] 1.2.0 release to develop [incubator-sdap-nexus]

2024-03-18 Thread via GitHub
RKuttruff opened a new pull request, #304: URL: https://github.com/apache/incubator-sdap-nexus/pull/304 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[PR] Forward 1.2.0 release to develop [incubator-sdap-nexus]

2024-03-15 Thread via GitHub
RKuttruff opened a new pull request, #303: URL: https://github.com/apache/incubator-sdap-nexus/pull/303 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] Forward 1.2.0 release to develop [incubator-sdap-ingester]

2024-03-15 Thread via GitHub
RKuttruff closed pull request #98: Forward 1.2.0 release to develop URL: https://github.com/apache/incubator-sdap-ingester/pull/98 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[PR] Forward 1.2.0 release to develop [incubator-sdap-ingester]

2024-03-15 Thread via GitHub
RKuttruff opened a new pull request, #98: URL: https://github.com/apache/incubator-sdap-ingester/pull/98 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] Release/1.2.0 [incubator-sdap-ingester]

2024-03-15 Thread via GitHub
skorper merged PR #88: URL: https://github.com/apache/incubator-sdap-ingester/pull/88 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Fixed install of kubectl in Collection Manager image build [incubator-sdap-ingester]

2024-03-15 Thread via GitHub
skorper closed pull request #96: Fixed install of kubectl in Collection Manager image build URL: https://github.com/apache/incubator-sdap-ingester/pull/96 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] Release/1.2.0 [incubator-sdap-nexus]

2024-03-15 Thread via GitHub
skorper merged PR #278: URL: https://github.com/apache/incubator-sdap-nexus/pull/278 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Prep for 1.2.0 release [incubator-sdap-website]

2024-03-15 Thread via GitHub
RKuttruff merged PR #12: URL: https://github.com/apache/incubator-sdap-website/pull/12 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-512 - Have GI close ZK & Solr connections when done with writing to Solr [incubator-sdap-ingester]

2024-03-14 Thread via GitHub
RKuttruff commented on PR #97: URL: https://github.com/apache/incubator-sdap-ingester/pull/97#issuecomment-1997869714 It seems Cassandra connections have this issue too... Going to hold off this PR to address this -- This is an automated message from the Apache Git Service. To respond

[PR] Added some missing ASF headers and removed some unneeded files [incubator-sdap-nexus]

2024-03-11 Thread via GitHub
RKuttruff opened a new pull request, #302: URL: https://github.com/apache/incubator-sdap-nexus/pull/302 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] SDAP-472 - data-access overhaul to support multiple simultaneous data backends [incubator-sdap-nexus]

2024-03-07 Thread via GitHub
RKuttruff merged PR #294: URL: https://github.com/apache/incubator-sdap-nexus/pull/294 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-505: Support DOMS insitu API [incubator-sdap-nexus]

2024-03-06 Thread via GitHub
skorper merged PR #299: URL: https://github.com/apache/incubator-sdap-nexus/pull/299 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-505: Support DOMS insitu API [incubator-sdap-nexus]

2024-03-06 Thread via GitHub
skorper commented on PR #299: URL: https://github.com/apache/incubator-sdap-nexus/pull/299#issuecomment-1981691088 @ngachung Good news! It appears to have worked. Tested like so: ```bash localhost:8083/match_spark?primary=AVHRR_OI_L4_GHRSST_NCEI=ICOADS Release

Re: [PR] SDAP-505: Support DOMS insitu API [incubator-sdap-nexus]

2024-03-06 Thread via GitHub
skorper commented on code in PR #299: URL: https://github.com/apache/incubator-sdap-nexus/pull/299#discussion_r1515077127 ## analysis/webservice/algorithms/doms/config.py: ## @@ -14,39 +14,29 @@ # limitations under the License. INSITU_API_ENDPOINT =

Re: [PR] SDAP-505: Support DOMS insitu API [incubator-sdap-nexus]

2024-03-06 Thread via GitHub
ngachung commented on code in PR #299: URL: https://github.com/apache/incubator-sdap-nexus/pull/299#discussion_r1515067880 ## analysis/webservice/algorithms/doms/config.py: ## @@ -14,39 +14,29 @@ # limitations under the License. INSITU_API_ENDPOINT =

Re: [PR] SDAP-505: Support DOMS insitu API [incubator-sdap-nexus]

2024-03-06 Thread via GitHub
skorper commented on PR #299: URL: https://github.com/apache/incubator-sdap-nexus/pull/299#issuecomment-1981610295 > Looks good to me. I'm assuming the new NCAR endpoint re-adds insitu support after the CDMS project went down? Yes, we are now pointing to the DOMS insitu API FSU and

Re: [PR] SDAP-505: Support DOMS insitu API [incubator-sdap-nexus]

2024-03-06 Thread via GitHub
skorper commented on PR #299: URL: https://github.com/apache/incubator-sdap-nexus/pull/299#issuecomment-1981540473 Updated NCAR insitu api url, tested like so: ```bash localhost:8083/match_spark?primary=AVHRR_OI_L4_GHRSST_NCEI=ICOADS Release

[PR] SDAP-512 - Have GI close ZK & Solr connections when done with writing to Solr [incubator-sdap-ingester]

2024-03-06 Thread via GitHub
RKuttruff opened a new pull request, #97: URL: https://github.com/apache/incubator-sdap-ingester/pull/97 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[PR] Fixed install of kubectl in Collection Manager image build [incubator-sdap-ingester]

2024-03-05 Thread via GitHub
RKuttruff opened a new pull request, #96: URL: https://github.com/apache/incubator-sdap-ingester/pull/96 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
RKuttruff commented on PR #95: URL: https://github.com/apache/incubator-sdap-ingester/pull/95#issuecomment-1977727178 Merged. Thanks for your help with this transition! We should be all caught up to get RC4 started. -- This is an automated message from the Apache Git Service. To

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
RKuttruff merged PR #95: URL: https://github.com/apache/incubator-sdap-ingester/pull/95 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
skorper commented on PR #95: URL: https://github.com/apache/incubator-sdap-ingester/pull/95#issuecomment-1977725373 > > > > Shouldn't we also change collection manager? > > > > > > > > > We should, but I left it as-is for now, since it wasn't causing any issues re: GPL code >

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
RKuttruff commented on PR #95: URL: https://github.com/apache/incubator-sdap-ingester/pull/95#issuecomment-1977724928 > Same as nexus, I think we should remove VERSION.txt This would be dependent on whether CM switches over to using poetry too -- This is an automated message from

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
RKuttruff commented on PR #95: URL: https://github.com/apache/incubator-sdap-ingester/pull/95#issuecomment-1977724237 > > > Shouldn't we also change collection manager? > > > > > > We should, but I left it as-is for now, since it wasn't causing any issues re: GPL code > >

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
RKuttruff commented on code in PR #95: URL: https://github.com/apache/incubator-sdap-ingester/pull/95#discussion_r1511973018 ## granule_ingester/pyproject.toml: ## @@ -0,0 +1,59 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
skorper commented on PR #95: URL: https://github.com/apache/incubator-sdap-ingester/pull/95#issuecomment-1977722577 > > Shouldn't we also change collection manager? > > We should, but I left it as-is for now, since it wasn't causing any issues re: GPL code No time like the

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
skorper commented on PR #95: URL: https://github.com/apache/incubator-sdap-ingester/pull/95#issuecomment-1977721377 Same as nexus, I think we should remove VERSION.txt -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
skorper commented on code in PR #95: URL: https://github.com/apache/incubator-sdap-ingester/pull/95#discussion_r1511970533 ## granule_ingester/pyproject.toml: ## @@ -0,0 +1,59 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
RKuttruff commented on PR #95: URL: https://github.com/apache/incubator-sdap-ingester/pull/95#issuecomment-1977718584 > Shouldn't we also change collection manager? We should, but I left it as-is for now, since it wasn't causing any issues re: GPL code -- This is an automated

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
RKuttruff merged PR #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
RKuttruff commented on PR #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301#issuecomment-1977712682 > I think we should also remove VERSION.txt and manage the version solely in the pyproject.toml file Done -- This is an automated message from the Apache Git

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
skorper commented on PR #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301#issuecomment-1977707259 I think we should also remove VERSION.txt and manage the version solely in the pyproject.toml file -- This is an automated message from the Apache Git Service. To respond

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
RKuttruff commented on PR #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301#issuecomment-1977690230 I agree. Old files have been removed Ingester common & collection manager still have their setup.py, but they never used conda -- This is an automated message

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
skorper commented on PR #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301#issuecomment-1977680451 > > What about the setup.py files, requirements.txt files, conda requirements files etc? Those are still in the repo currently. > > Also, what are your thoughts about

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
RKuttruff commented on code in PR #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301#discussion_r1511932667 ## docker/nexus-webapp/entrypoint.sh: ## @@ -0,0 +1,133 @@ +#!/bin/bash Review Comment: Done -- This is an automated message from the Apache Git

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
RKuttruff commented on code in PR #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301#discussion_r1511931587 ## docker/nexus-webapp/entrypoint.sh: ## @@ -0,0 +1,133 @@ +#!/bin/bash Review Comment: Oh lol, that makes sense! I'll delete the .old It was

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
skorper commented on code in PR #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301#discussion_r1511929498 ## docker/nexus-webapp/entrypoint.sh: ## @@ -0,0 +1,133 @@ +#!/bin/bash Review Comment: What the heck, I swear I put this comment on Dockerfile.old

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
RKuttruff commented on code in PR #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301#discussion_r1511913900 ## pyproject.toml: ## @@ -0,0 +1,75 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
RKuttruff commented on code in PR #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301#discussion_r1511912292 ## docker/nexus-webapp/entrypoint.sh: ## @@ -0,0 +1,133 @@ +#!/bin/bash Review Comment: I think we may need this file. It was the entrypoint for the

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
RKuttruff commented on PR #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301#issuecomment-1977626124 > What about the setup.py files, requirements.txt files, conda requirements files etc? Those are still in the repo currently. > > Also, what are your thoughts about

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
skorper commented on code in PR #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301#discussion_r1511902837 ## pyproject.toml: ## @@ -0,0 +1,75 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the

[PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
RKuttruff opened a new pull request, #301: URL: https://github.com/apache/incubator-sdap-nexus/pull/301 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-04 Thread via GitHub
RKuttruff closed pull request #300: SDAP-511 - Switch package manager to poetry URL: https://github.com/apache/incubator-sdap-nexus/pull/300 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
RKuttruff opened a new pull request, #95: URL: https://github.com/apache/incubator-sdap-ingester/pull/95 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] SDAP-483: Added `.asf.yaml` to configure Jira auto-linking. [incubator-sdap-nexusproto]

2024-03-04 Thread via GitHub
RKuttruff closed pull request #17: SDAP-483: Added `.asf.yaml` to configure Jira auto-linking. URL: https://github.com/apache/incubator-sdap-nexusproto/pull/17 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] Added ASF yaml [incubator-sdap-nexusproto]

2024-03-04 Thread via GitHub
RKuttruff merged PR #20: URL: https://github.com/apache/incubator-sdap-nexusproto/pull/20 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Updated ASF yaml [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
RKuttruff merged PR #94: URL: https://github.com/apache/incubator-sdap-ingester/pull/94 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[PR] Updated ASF yaml [incubator-sdap-ingester]

2024-03-04 Thread via GitHub
RKuttruff opened a new pull request, #94: URL: https://github.com/apache/incubator-sdap-ingester/pull/94 - Branch protections - Branch auto-delete -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[PR] Added ASF yaml [incubator-sdap-nexusproto]

2024-03-04 Thread via GitHub
RKuttruff opened a new pull request, #20: URL: https://github.com/apache/incubator-sdap-nexusproto/pull/20 - Branch protections - Branch auto-delete - Jira auto-linking -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Handle ingestion of previously ingested files [incubator-sdap-in-situ-data-services]

2024-03-01 Thread via GitHub
RKuttruff closed pull request #26: Handle ingestion of previously ingested files URL: https://github.com/apache/incubator-sdap-in-situ-data-services/pull/26 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] Data validation tool lambda [incubator-sdap-in-situ-data-services]

2024-03-01 Thread via GitHub
RKuttruff merged PR #24: URL: https://github.com/apache/incubator-sdap-in-situ-data-services/pull/24 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Indexing Lambda fix [incubator-sdap-in-situ-data-services]

2024-03-01 Thread via GitHub
RKuttruff merged PR #27: URL: https://github.com/apache/incubator-sdap-in-situ-data-services/pull/27 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[PR] SDAP-511 - Switch package manager to poetry [incubator-sdap-nexus]

2024-03-01 Thread via GitHub
RKuttruff opened a new pull request, #300: URL: https://github.com/apache/incubator-sdap-nexus/pull/300 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[PR] Indexing Lambda fix [incubator-sdap-in-situ-data-services]

2024-03-01 Thread via GitHub
RKuttruff opened a new pull request, #27: URL: https://github.com/apache/incubator-sdap-in-situ-data-services/pull/27 When going through a SQS messages of S3 events, indexer would break the loop when a temp parquet file was encountered. If a valid parquet file event followed in the

[PR] SDAP-505: Support DOMS insitu API [incubator-sdap-nexus]

2024-02-15 Thread via GitHub
skorper opened a new pull request, #299: URL: https://github.com/apache/incubator-sdap-nexus/pull/299 - Cast platform to string. SDAP requires platform to be a string (db col is str) but the DOMS insitu API seems to return an integer. - Add mechanism for not supporting insitu schema. As

Re: [PR] Update quickstart with new solr command [incubator-sdap-nexus]

2024-02-15 Thread via GitHub
skorper commented on PR #298: URL: https://github.com/apache/incubator-sdap-nexus/pull/298#issuecomment-1947579156 I can try to recreate the issue a bit later. But in summary, the solr container failed to launch because it could not find the zookeeper host. It seemed like `ZK_HOST` was

Re: [PR] Update quickstart with new solr command [incubator-sdap-nexus]

2024-02-15 Thread via GitHub
RKuttruff commented on PR #298: URL: https://github.com/apache/incubator-sdap-nexus/pull/298#issuecomment-1947524926 This is odd, my existing deployment with Docker uses `ZK_HOST` without issue. What kind of error were you seeing? -- This is an automated message from the Apache Git

[PR] Update quickstart with new solr command [incubator-sdap-nexus]

2024-02-15 Thread via GitHub
skorper opened a new pull request, #298: URL: https://github.com/apache/incubator-sdap-nexus/pull/298 I was running through the quickstart and found the `ZK_HOST` environment variable wasn't being used by the solr container, resulting in failure. By updating to `SDAP_ZK_SERVICE_HOST` the

[PR] SDAP-495 - ABoVE Lidar Vizualization [incubator-sdap-nexus]

2024-02-08 Thread via GitHub
RKuttruff opened a new pull request, #297: URL: https://github.com/apache/incubator-sdap-nexus/pull/297 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] SDAP-508 - Report satellite dataset spatial extents in list endpoints [incubator-sdap-nexus]

2024-02-06 Thread via GitHub
skorper merged PR #295: URL: https://github.com/apache/incubator-sdap-nexus/pull/295 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] domspurge patch [incubator-sdap-nexus]

2024-02-06 Thread via GitHub
skorper merged PR #280: URL: https://github.com/apache/incubator-sdap-nexus/pull/280 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] SDAP-498 - Satellite Units [incubator-sdap-nexus]

2024-02-06 Thread via GitHub
RKuttruff commented on PR #296: URL: https://github.com/apache/incubator-sdap-nexus/pull/296#issuecomment-1930947532 Draft pending #294 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

  1   2   3   4   5   >