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

discard metadata URL based on their hash, not URLs #1810

Merged
merged 1 commit into from
Jun 28, 2020

Conversation

KtorZ
Copy link
Member

@KtorZ KtorZ commented Jun 27, 2020

Issue Number

#1805

Overview

  • I have changed the SQLite filtering to discard metadata URL based on their hash, not URLs

@KtorZ KtorZ requested a review from disassembler June 27, 2020 19:12
@KtorZ KtorZ self-assigned this Jun 27, 2020
@KtorZ KtorZ added the RESOLVING ISSUE Mark a PR as resolving issues, for auto-generated CHANGELOG label Jun 27, 2020
@KtorZ KtorZ force-pushed the KtorZ/fix-metadata-blacklisting branch from 149c2a0 to 2f0aa23 Compare June 27, 2020 19:36
@disassembler
Copy link
Contributor

tested and this shows stake pools with multiple registrations right away!

@KtorZ
Copy link
Member Author

KtorZ commented Jun 27, 2020

bors r+

iohk-bors bot added a commit that referenced this pull request Jun 27, 2020
1810: discard metadata URL based on their hash, not URLs r=KtorZ a=KtorZ

# Issue Number

<!-- Put here a reference to the issue this PR relates to and which requirements it tackles -->

#1805 

# Overview

<!-- Detail in a few bullet points the work accomplished in this PR -->

- [x] I have changed the SQLite filtering to discard metadata URL based on their hash, not URLs


<!-- 
Don't forget to:

 ✓ Self-review your changes to make sure nothing unexpected slipped through
 ✓ Assign yourself to the PR
 ✓ Assign one or several reviewer(s)
 ✓ Once created, link this PR to its corresponding ticket
 ✓ Assign the PR to a corresponding milestone
 ✓ Acknowledge any changes required to the Wiki
-->


Co-authored-by: KtorZ <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jun 27, 2020

Build failed

@KtorZ
Copy link
Member Author

KtorZ commented Jun 27, 2020

bors retry

iohk-bors bot added a commit that referenced this pull request Jun 27, 2020
1810: discard metadata URL based on their hash, not URLs r=KtorZ a=KtorZ

# Issue Number

<!-- Put here a reference to the issue this PR relates to and which requirements it tackles -->

#1805 

# Overview

<!-- Detail in a few bullet points the work accomplished in this PR -->

- [x] I have changed the SQLite filtering to discard metadata URL based on their hash, not URLs


<!-- 
Don't forget to:

 ✓ Self-review your changes to make sure nothing unexpected slipped through
 ✓ Assign yourself to the PR
 ✓ Assign one or several reviewer(s)
 ✓ Once created, link this PR to its corresponding ticket
 ✓ Assign the PR to a corresponding milestone
 ✓ Acknowledge any changes required to the Wiki
-->


Co-authored-by: KtorZ <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jun 27, 2020

Build failed

@KtorZ KtorZ force-pushed the KtorZ/fix-metadata-blacklisting branch from 2f0aa23 to c702f82 Compare June 27, 2020 22:07
@KtorZ
Copy link
Member Author

KtorZ commented Jun 27, 2020

bors r+

iohk-bors bot added a commit that referenced this pull request Jun 27, 2020
1810: discard metadata URL based on their hash, not URLs r=KtorZ a=KtorZ

# Issue Number

<!-- Put here a reference to the issue this PR relates to and which requirements it tackles -->

#1805 

# Overview

<!-- Detail in a few bullet points the work accomplished in this PR -->

- [x] I have changed the SQLite filtering to discard metadata URL based on their hash, not URLs


<!-- 
Don't forget to:

 ✓ Self-review your changes to make sure nothing unexpected slipped through
 ✓ Assign yourself to the PR
 ✓ Assign one or several reviewer(s)
 ✓ Once created, link this PR to its corresponding ticket
 ✓ Assign the PR to a corresponding milestone
 ✓ Acknowledge any changes required to the Wiki
-->


Co-authored-by: KtorZ <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jun 27, 2020

Build failed

@rvl
Copy link
Contributor

rvl commented Jun 28, 2020

I opened issue #1811 about the test which just failed in bors - retrying now.

bors r+

iohk-bors bot added a commit that referenced this pull request Jun 28, 2020
1810: discard metadata URL based on their hash, not URLs r=rvl a=KtorZ

# Issue Number

<!-- Put here a reference to the issue this PR relates to and which requirements it tackles -->

#1805 

# Overview

<!-- Detail in a few bullet points the work accomplished in this PR -->

- [x] I have changed the SQLite filtering to discard metadata URL based on their hash, not URLs


<!-- 
Don't forget to:

 ✓ Self-review your changes to make sure nothing unexpected slipped through
 ✓ Assign yourself to the PR
 ✓ Assign one or several reviewer(s)
 ✓ Once created, link this PR to its corresponding ticket
 ✓ Assign the PR to a corresponding milestone
 ✓ Acknowledge any changes required to the Wiki
-->


Co-authored-by: KtorZ <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jun 28, 2020

Build failed

@rvl
Copy link
Contributor

rvl commented Jun 28, 2020

Doh, forgot that the test result is cached - restarted the test build in Hydra.

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jun 28, 2020

@iohk-bors iohk-bors bot merged commit f6d5a80 into master Jun 28, 2020
@iohk-bors iohk-bors bot deleted the KtorZ/fix-metadata-blacklisting branch June 28, 2020 01:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RESOLVING ISSUE Mark a PR as resolving issues, for auto-generated CHANGELOG
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants