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

CI release bug fixes #1611

Merged
merged 10 commits into from
Nov 21, 2023
Merged

CI release bug fixes #1611

merged 10 commits into from
Nov 21, 2023

Conversation

gpmayorga
Copy link
Contributor

@gpmayorga gpmayorga commented Nov 20, 2023

Description

Quick fixes testing CI pipeline on main branch:

  1. Bad conditional never publishes Docker container to DH

  2. Publishing docs using an already old action method. Migrated to the new official GitHub action

wischli
wischli previously approved these changes Nov 20, 2023
Copy link
Contributor

@wischli wischli left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@gpmayorga gpmayorga changed the title Fix main branch detection for docker build CI release bug fixes Nov 20, 2023
@gpmayorga
Copy link
Contributor Author

@wischli this should be ready to go

@gpmayorga gpmayorga enabled auto-merge (squash) November 20, 2023 15:52
@gpmayorga gpmayorga added I2-bug The code fails to follow expected behaviour. Q0-trivial An issure which is similar to patching code. D0-ready Pull request can be merged without special precaution and notification. P7-asap Issue should be addressed in the next days. I12-ci Issue related to CI github_actions Pull requests that update GitHub Actions code and removed P7-asap Issue should be addressed in the next days. labels Nov 20, 2023
@gpmayorga gpmayorga merged commit 6330858 into main Nov 21, 2023
11 of 14 checks passed
@gpmayorga gpmayorga deleted the ci_bug_fixes branch November 21, 2023 12:12
wischli pushed a commit that referenced this pull request Nov 22, 2023
* Fix main branch detection for docker build

* Try out GHA for Pages deployment

* Docker build clean up workflow

* publish docker in all non-PR events

* fix docs cache (github token)

* Deploy docs only from main branch

* revert docker build machine size
@wischli wischli mentioned this pull request Nov 24, 2023
4 tasks
wischli added a commit that referenced this pull request Dec 7, 2023
* feat: new weights altair

* feat: new weights centrifuge

* wip

* migrations: Add migration for account data

* migrations: Adjust pre upgrade logic for accounts

* migrations: Log total count for accounts

* migrations: Log total count for accounts with frozen and reserved balance

* deps: Remove double orml-tokens dep

* migrations: Add test account data for balances

* fix: account info checks

* fix: local docker setup

* CI release bug fixes  (#1611)

* Fix main branch detection for docker build

* Try out GHA for Pages deployment

* Docker build clean up workflow

* publish docker in all non-PR events

* fix docs cache (github token)

* Deploy docs only from main branch

* revert docker build machine size

* Fix: Transfer debt (#1615)

* fix: blocking transfer debt by epoch and redemptions.

* feat: make change_id deterministic

* fix: build

* chore(deps): bump google-github-actions/auth from 1.1.1 to 1.2.0 (#1614)

Bumps [google-github-actions/auth](https://github.com/google-github-actions/auth) from 1.1.1 to 1.2.0.
- [Release notes](https://github.com/google-github-actions/auth/releases)
- [Changelog](https://github.com/google-github-actions/auth/blob/main/CHANGELOG.md)
- [Commits](google-github-actions/auth@35b0e87...f105ef0)

---
updated-dependencies:
- dependency-name: google-github-actions/auth
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Guillermo Perez <gpmayorga@users.noreply.github.com>

* fix: minor stuff balances migration

* chore: bump development version, rm dummy migration

* chore: bump MaxFeedValues from 10 to 500

* fix: toml versions

* fix: remove balance migration from runtimes

* refactor: remove balance migration from codebase

* chore: update dev weights

* chore: update cfg weights

* chore: altair weights

* fix: enable XCM benches for dev

* chore: bump dev to 1035

chore: bump lock

* fix: missed renaming

fix: weights

fix: allowlist weights

* chore: bump client to 0.10.35

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: cdamian <17934949+cdamian@users.noreply.github.com>
Co-authored-by: William Freudenberger <w.freude@icloud.com>
Co-authored-by: Guillermo Perez <gpmayorga@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D0-ready Pull request can be merged without special precaution and notification. github_actions Pull requests that update GitHub Actions code I2-bug The code fails to follow expected behaviour. I12-ci Issue related to CI Q0-trivial An issure which is similar to patching code.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants