Skip to content

spread: add external:ubuntu-core-24-64#12345

Merged
sergiocazzolato merged 1 commit intocanonical:masterfrom
valentindavid:valentindavid/external-ubuntu-core-24-64
Jan 25, 2023
Merged

spread: add external:ubuntu-core-24-64#12345
sergiocazzolato merged 1 commit intocanonical:masterfrom
valentindavid:valentindavid/external-ubuntu-core-24-64

Conversation

@valentindavid
Copy link
Contributor

This is needed by core-base CI.

@valentindavid
Copy link
Contributor Author

Needed by canonical/core-base#89

Copy link
Contributor

@mvo5 mvo5 left a comment

Choose a reason for hiding this comment

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

Thank you, this looks fine except the one debug leftover. AFAICT we could promote this out of draft and merge, or is there a downside of doing this?

Copy link
Contributor

Choose a reason for hiding this comment

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

This looks like a debug leftover

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it is still in use. I will remove it before I make it ready for review.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have removed it.

Copy link
Contributor

@sergiocazzolato sergiocazzolato left a comment

Choose a reason for hiding this comment

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

just a comment about moving part of the change to snapd-testing-tools project

Copy link
Contributor

Choose a reason for hiding this comment

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

Hi, this needs to be included in the snapd-testing-tools project

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

I think you just need to merge master because it is on master

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@valentindavid valentindavid force-pushed the valentindavid/external-ubuntu-core-24-64 branch from de7785a to b9a79b3 Compare December 2, 2022 15:42
@valentindavid valentindavid marked this pull request as ready for review December 2, 2022 15:53
@valentindavid valentindavid force-pushed the valentindavid/external-ubuntu-core-24-64 branch from b9a79b3 to 2e902a8 Compare December 8, 2022 10:44
Copy link
Contributor

@mvo5 mvo5 left a comment

Choose a reason for hiding this comment

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

Thank you

Copy link
Contributor

@sergiocazzolato sergiocazzolato left a comment

Choose a reason for hiding this comment

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

Hi, I left a comment about os.query is-core24

@valentindavid valentindavid force-pushed the valentindavid/external-ubuntu-core-24-64 branch 2 times, most recently from fd43176 to e8b879f Compare January 9, 2023 10:28
This is needed by core-base CI.
@valentindavid valentindavid force-pushed the valentindavid/external-ubuntu-core-24-64 branch from e8b879f to 6d2d5c3 Compare January 23, 2023 13:32
Copy link
Contributor

@sergiocazzolato sergiocazzolato left a comment

Choose a reason for hiding this comment

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

Nice, thanks for this

@sergiocazzolato sergiocazzolato merged commit 0766efa into canonical:master Jan 25, 2023
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