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.

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

Set up machinery for testing HTTP interactions of HTTP registry #818

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

mkaput
Copy link
Member

@mkaput mkaput commented Oct 23, 2023

github-merge-queue bot pushed a commit that referenced this pull request Oct 25, 2023
This implementation focuses solely on local registries, and it has
almost no security.

---

**Stack**:
- #819
- #818
- #809
- #808
- #807
- #806
- #805
- #803
- #802
- #799
- #798
- #793
- #790⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
@mkaput mkaput force-pushed the spr/main/ca3e5f93 branch from e854b6f to f5d20ce Compare October 25, 2023 12:21
github-merge-queue bot pushed a commit that referenced this pull request Oct 25, 2023
There are some rough untested edges, but in general this command is
capable of building a local registry from scratch 🎉

---

**Stack**:
- #819
- #818
- #809
- #808
- #807
- #806
- #805
- #803
- #802
- #799
- #798
- #793⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
@mkaput mkaput force-pushed the spr/main/97fc578f branch from 95e0a46 to d293563 Compare October 27, 2023 12:44
@mkaput mkaput force-pushed the spr/main/ca3e5f93 branch from f5d20ce to 3b300c3 Compare October 27, 2023 12:44
github-merge-queue bot pushed a commit that referenced this pull request Oct 27, 2023
These tests depend on both #790 and #793, hence they come in separate
PR.

---

**Stack**:
- #819
- #818
- #809
- #808
- #807
- #806
- #805
- #803
- #802
- #799
- #798⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
@mkaput mkaput force-pushed the spr/main/97fc578f branch from d293563 to 8268aa2 Compare October 27, 2023 13:24
@mkaput mkaput force-pushed the spr/main/ca3e5f93 branch from 3b300c3 to 1529ca2 Compare October 27, 2023 13:24
github-merge-queue bot pushed a commit that referenced this pull request Oct 27, 2023
**Stack**:
- #819
- #818
- #809
- #808
- #807
- #806
- #805
- #803
- #802
- #799⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
@mkaput mkaput force-pushed the spr/main/ca3e5f93 branch from 1529ca2 to cfe44d5 Compare October 27, 2023 14:16
@mkaput mkaput force-pushed the spr/main/97fc578f branch from 8268aa2 to b7b62c1 Compare October 27, 2023 14:16
github-merge-queue bot pushed a commit that referenced this pull request Oct 27, 2023
**Stack**:
- #819
- #818
- #809
- #808
- #807
- #806
- #805
- #803
- #802⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
@mkaput mkaput force-pushed the spr/main/97fc578f branch from b7b62c1 to b039be7 Compare October 27, 2023 14:51
@mkaput mkaput force-pushed the spr/main/ca3e5f93 branch from cfe44d5 to 5aa2ddc Compare October 27, 2023 14:51
github-merge-queue bot pushed a commit that referenced this pull request Oct 27, 2023
**Stack**:
- #819
- #818
- #809
- #808
- #807
- #806
- #805
- #803⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
@mkaput mkaput force-pushed the spr/main/97fc578f branch from b039be7 to 74e0606 Compare October 27, 2023 15:52
@mkaput mkaput force-pushed the spr/main/ca3e5f93 branch from 5aa2ddc to 1ee6e2f Compare October 27, 2023 15:52
github-merge-queue bot pushed a commit that referenced this pull request Oct 27, 2023
**Stack**:
- #819
- #818
- #809
- #808
- #807
- #806
- #805⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
@mkaput mkaput force-pushed the spr/main/ca3e5f93 branch from 0f9d6b5 to bd285e4 Compare October 30, 2023 17:56
github-merge-queue bot pushed a commit that referenced this pull request Oct 30, 2023
**Stack**:
- #846
- #845
- #844
- #843
- #819
- #818
- #809
- #808
- #807⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
@mkaput mkaput force-pushed the spr/main/97fc578f branch from 12c9170 to 5b767e2 Compare October 31, 2023 08:19
@mkaput mkaput force-pushed the spr/main/ca3e5f93 branch from bd285e4 to 2cb880a Compare October 31, 2023 08:19
@mkaput mkaput force-pushed the spr/main/97fc578f branch from 5b767e2 to 89b884a Compare November 7, 2023 14:26
@mkaput mkaput force-pushed the spr/main/ca3e5f93 branch from 2cb880a to b62b9f8 Compare November 7, 2023 14:27
@mkaput mkaput force-pushed the spr/main/97fc578f branch from 89b884a to d2e41f1 Compare November 8, 2023 08:57
@mkaput mkaput force-pushed the spr/main/ca3e5f93 branch from b62b9f8 to 423eb95 Compare November 8, 2023 08:57
@mkaput mkaput requested review from maciektr and szymmis November 8, 2023 08:59
@mkaput mkaput force-pushed the spr/main/ca3e5f93 branch from 423eb95 to e7b3c11 Compare November 8, 2023 10:39
@mkaput mkaput force-pushed the spr/main/97fc578f branch from d2e41f1 to a9a4b50 Compare November 8, 2023 10:39
@mkaput mkaput mentioned this pull request Nov 9, 2023
@mkaput mkaput force-pushed the spr/main/ca3e5f93 branch from e7b3c11 to c2e84ea Compare November 9, 2023 15:25
github-merge-queue bot pushed a commit that referenced this pull request Nov 13, 2023
This PR lays groundwork for implementing caching of `RegistryClient`
instances.
It just sets up all the types and refactors, no caching is done yet.

There is also a small functional change: local registry client does not
print
`Unpacking` status in normal mode. This wasn't a big deal, and it
allowed removing
the `is_offline` method from `RegistryClient`.

---

**Stack**:
- #892
- #846
- #845
- #844
- #819
- #818
- #809
- #808⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*

Signed-off-by: Marek Kaput <marek.kaput@swmansion.com>
github-merge-queue bot pushed a commit that referenced this pull request Nov 13, 2023
NOTE: Nothing is cached actually yet, because no registry client uses
hooks provided by this code. This will come in subsequent PRs.

---

**Stack**:
- #892
- #846
- #845
- #844
- #819
- #818
- #809⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
Base automatically changed from spr/main/97fc578f to main November 13, 2023 13:29
Signed-off-by: Marek Kaput <marek.kaput@swmansion.com>

commit-id:ca3e5f93
@mkaput mkaput enabled auto-merge November 13, 2023 13:30
@mkaput mkaput added this pull request to the merge queue Nov 13, 2023
Merged via the queue into main with commit ecebc03 Nov 13, 2023
11 checks passed
@mkaput mkaput deleted the spr/main/ca3e5f93 branch November 13, 2023 14:01
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.

3 participants