Merge remote-tracking branch 'origin/main' into sanaz/backport-optimi…

…zation-knuu

.github/workflows/create_release_tracking_epic.yml

@@ -7,7 +7,7 @@ on:
     types: [released]
 jobs:
   trigger_issue:
-    uses: celestiaorg/.github/.github/workflows/[email protected].2
+    uses: celestiaorg/.github/.github/workflows/[email protected].3
     secrets: inherit
     with:
       release-repo: ${{ github.repository }}

.github/workflows/docker-build-publish.yml

@@ -20,7 +20,7 @@ jobs:
     permissions:
       contents: write
       packages: write
-    uses: celestiaorg/.github/.github/workflows/[email protected].2
+    uses: celestiaorg/.github/.github/workflows/[email protected].3
     with:
       dockerfile: Dockerfile
     secrets: inherit
@@ -29,7 +29,7 @@ jobs:
     permissions:
       contents: write
       packages: write
-    uses: celestiaorg/.github/.github/workflows/[email protected].2
+    uses: celestiaorg/.github/.github/workflows/[email protected].3
     with:
       dockerfile: docker/Dockerfile_txsim
       packageName: txsim

.github/workflows/goreleaser.yml

@@ -11,7 +11,7 @@ jobs:
     steps:
       - name: checkout
         uses: actions/checkout@v4
-      - uses: goreleaser/goreleaser-action@v5
+      - uses: goreleaser/goreleaser-action@v6
         with:
           version: latest
           args: check

.github/workflows/lint.yml

@@ -30,10 +30,10 @@ jobs:
 
   # hadolint lints the Dockerfile
   hadolint:
-    uses: celestiaorg/.github/.github/workflows/[email protected].2
+    uses: celestiaorg/.github/.github/workflows/[email protected].3
 
   yamllint:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: celestiaorg/.github/.github/actions/[email protected].2
+      - uses: celestiaorg/.github/.github/actions/[email protected].3

.github/workflows/pr-review-requester.yml

@@ -11,7 +11,7 @@ on:
 jobs:
   auto-request-review:
     name: Auto request reviews
-    uses: celestiaorg/.github/.github/workflows/[email protected].2
+    uses: celestiaorg/.github/.github/workflows/[email protected].3
     secrets: inherit
     # write access for issues and pull requests is needed because the called
     # workflow requires write access to issues and pull requests and the

.github/workflows/test-e2e.yml

@@ -1,10 +1,13 @@
 name: test-e2e
 
 on:
-  workflow_call:
-    secrets:
-      KNUU_KUBECONFIG_FILE:
-        required: true
+  push:
+    branches:
+      - main
+    paths:
+      - "**.go"
+      - go.mod
+      - go.sum
 
 jobs:
   test-e2e:
@@ -25,3 +28,14 @@ jobs:
           echo "${KUBECONFIG_FILE}" > $HOME/.kube/config
       - name: Run e2e tests
         run: make test-e2e
+
+      - name: If the e2e test fails, notify Slack channel #e2e-test-failures
+        if: failure()
+        uses: ravsamhq/notify-slack-action@v2
+        with:
+          status: ${{ job.status }}
+          token: ${{ secrets.GITHUB_TOKEN }}
+          notification_title: "E2E test failure"
+          message_format: "{emoji} *{workflow}* {status_message} in <{run_url}|{repo}>"
+        env:
+          SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_E2E_TEST_FAILURES }}

.github/workflows/test.yml

@@ -41,7 +41,7 @@ jobs:
         run: make test-coverage
 
       - name: Upload coverage.txt
-        uses: codecov/codecov-action@v4.4.1
+        uses: codecov/codecov-action@v4.5.0
         with:
           file: ./coverage.txt
 

.goreleaser.yaml

@@ -2,6 +2,7 @@
 # Since GoReleaser doesn't support CGO natively, our GoReleaser process builds
 # binaries in a Docker image maintained by goreleaser-cross that has CGO support
 # for multiple platforms. See https://github.com/goreleaser/goreleaser-cross
+version: 2
 before:
   hooks:
     - go mod tidy

Makefile

@@ -213,8 +213,3 @@ prebuilt-binary:
 		ghcr.io/goreleaser/goreleaser-cross:${GOLANG_CROSS_VERSION} \
 		release --clean
 .PHONY: prebuilt-binary
-
-## govulncheck: Check for vulnerabilities in dependencies.
-govulncheck:
-	@go run golang.org/x/vuln/cmd/[email protected] ./...
-.PHONY: govulncheck

app/ante/fee_checker.go

@@ -27,7 +27,7 @@ func ValidateTxFeeWrapper(paramKeeper params.Keeper) ante.TxFeeChecker {
 
 // ValidateTxFee implements default fee validation logic for transactions.
 // It ensures that the provided transaction fee meets a minimum threshold for the node
-// as well as a global minimum threshold and computes the tx priority based on the gas price.
+// as well as a network minimum threshold and computes the tx priority based on the gas price.
 func ValidateTxFee(ctx sdk.Context, tx sdk.Tx, paramKeeper params.Keeper) (sdk.Coins, int64, error) {
 	feeTx, ok := tx.(sdk.FeeTx)
 	if !ok {
@@ -50,24 +50,24 @@ func ValidateTxFee(ctx sdk.Context, tx sdk.Tx, paramKeeper params.Keeper) (sdk.C
 		}
 	}
 
-	// Ensure that the provided fee meets a global minimum threshold.
-	// Global minimum fee only applies to app versions greater than one
+	// Ensure that the provided fee meets a network minimum threshold.
+	// Network minimum fee only applies to app versions greater than one.
 	if ctx.BlockHeader().Version.App > v1.Version {
 		subspace, exists := paramKeeper.GetSubspace(minfee.ModuleName)
 		if !exists {
 			return nil, 0, errors.Wrap(sdkerror.ErrInvalidRequest, "minfee is not a registered subspace")
 		}
 
-		if !subspace.Has(ctx, minfee.KeyGlobalMinGasPrice) {
-			return nil, 0, errors.Wrap(sdkerror.ErrKeyNotFound, "GlobalMinGasPrice")
+		if !subspace.Has(ctx, minfee.KeyNetworkMinGasPrice) {
+			return nil, 0, errors.Wrap(sdkerror.ErrKeyNotFound, "NetworkMinGasPrice")
 		}
 
-		var globalMinGasPrice sdk.Dec
-		// Gets the global minimum gas price from the param store
-		// panics if not configured properly
-		subspace.Get(ctx, minfee.KeyGlobalMinGasPrice, &globalMinGasPrice)
+		var networkMinGasPrice sdk.Dec
+		// Gets the network minimum gas price from the param store.
+		// Panics if not configured properly.
+		subspace.Get(ctx, minfee.KeyNetworkMinGasPrice, &networkMinGasPrice)
 
-		err := verifyMinFee(fee, gas, globalMinGasPrice, "insufficient gas price for the network")
+		err := verifyMinFee(fee, gas, networkMinGasPrice, "insufficient gas price for the network")
 		if err != nil {
 			return nil, 0, err
 		}

app/ante/min_fee_test.go

@@ -26,7 +26,7 @@ import (
 	tmdb "github.com/tendermint/tm-db"
 )
 
-func TestCheckTxFeeWithGlobalMinGasPrices(t *testing.T) {
+func TestValidateTxFee(t *testing.T) {
 	encCfg := encoding.MakeConfig(app.ModuleEncodingRegisters...)
 
 	builder := encCfg.TxConfig.NewTxBuilder()
@@ -58,31 +58,31 @@ func TestCheckTxFeeWithGlobalMinGasPrices(t *testing.T) {
 		{
 			name:       "bad tx; fee below required minimum",
 			fee:        sdk.NewCoins(sdk.NewInt64Coin(appconsts.BondDenom, feeAmount-1)),
-			gasLimit:   uint64(float64(feeAmount) / v2.GlobalMinGasPrice),
+			gasLimit:   uint64(float64(feeAmount) / v2.NetworkMinGasPrice),
 			appVersion: uint64(2),
 			isCheckTx:  false,
 			expErr:     true,
 		},
 		{
 			name:       "good tx; fee equal to required minimum",
 			fee:        sdk.NewCoins(sdk.NewInt64Coin(appconsts.BondDenom, feeAmount)),
-			gasLimit:   uint64(float64(feeAmount) / v2.GlobalMinGasPrice),
+			gasLimit:   uint64(float64(feeAmount) / v2.NetworkMinGasPrice),
 			appVersion: uint64(2),
 			isCheckTx:  false,
 			expErr:     false,
 		},
 		{
 			name:       "good tx; fee above required minimum",
 			fee:        sdk.NewCoins(sdk.NewInt64Coin(appconsts.BondDenom, feeAmount+1)),
-			gasLimit:   uint64(float64(feeAmount) / v2.GlobalMinGasPrice),
+			gasLimit:   uint64(float64(feeAmount) / v2.NetworkMinGasPrice),
 			appVersion: uint64(2),
 			isCheckTx:  false,
 			expErr:     false,
 		},
 		{
 			name:       "good tx; with no fee (v1)",
 			fee:        sdk.NewCoins(sdk.NewInt64Coin(appconsts.BondDenom, feeAmount)),
-			gasLimit:   uint64(float64(feeAmount) / v2.GlobalMinGasPrice),
+			gasLimit:   uint64(float64(feeAmount) / v2.NetworkMinGasPrice),
 			appVersion: uint64(1),
 			isCheckTx:  false,
 			expErr:     false,
@@ -143,12 +143,12 @@ func TestCheckTxFeeWithGlobalMinGasPrices(t *testing.T) {
 
 			ctx = ctx.WithMinGasPrices(sdk.DecCoins{validatorMinGasPriceCoin})
 
-			globalminGasPriceDec, err := sdk.NewDecFromStr(fmt.Sprintf("%f", v2.GlobalMinGasPrice))
+			networkMinGasPriceDec, err := sdk.NewDecFromStr(fmt.Sprintf("%f", v2.NetworkMinGasPrice))
 			require.NoError(t, err)
 
 			subspace, _ := paramsKeeper.GetSubspace(minfee.ModuleName)
 			subspace = minfee.RegisterMinFeeParamTable(subspace)
-			subspace.Set(ctx, minfee.KeyGlobalMinGasPrice, globalminGasPriceDec)
+			subspace.Set(ctx, minfee.KeyNetworkMinGasPrice, networkMinGasPriceDec)
 
 			_, _, err = ante.ValidateTxFee(ctx, tx, paramsKeeper)
 			if tc.expErr {
@@ -173,7 +173,7 @@ func setUp(t *testing.T) (paramkeeper.Keeper, storetypes.CommitMultiStore) {
 
 	registry := codectypes.NewInterfaceRegistry()
 
-	// Create a params keeper and set the global min gas price
+	// Create a params keeper and set the network min gas price.
 	paramsKeeper := paramkeeper.NewKeeper(codec.NewProtoCodec(registry), codec.NewLegacyAmino(), storeKey, tStoreKey)
 	paramsKeeper.Subspace(minfee.ModuleName)
 	return paramsKeeper, stateStore

app/app.go

@@ -301,13 +301,13 @@ func New(
 
 	paramBlockList := paramfilter.NewParamBlockList(app.BlockedParams()...)
 
-	// register the proposal types
+	// Register the proposal types.
 	govRouter := oldgovtypes.NewRouter()
 	govRouter.AddRoute(paramproposal.RouterKey, paramBlockList.GovHandler(app.ParamsKeeper)).
 		AddRoute(distrtypes.RouterKey, distr.NewCommunityPoolSpendProposalHandler(app.DistrKeeper)).
 		AddRoute(ibcclienttypes.RouterKey, NewClientProposalHandler(app.IBCKeeper.ClientKeeper))
 
-	// Create Transfer Keepers
+	// Create Transfer Keepers.
 	tokenFilterKeeper := tokenfilter.NewKeeper(app.IBCKeeper.ChannelKeeper)
 
 	app.PacketForwardKeeper = packetforwardkeeper.NewKeeper(
@@ -326,7 +326,7 @@ func New(
 		app.PacketForwardKeeper, app.IBCKeeper.ChannelKeeper, &app.IBCKeeper.PortKeeper,
 		app.AccountKeeper, app.BankKeeper, app.ScopedTransferKeeper,
 	)
-	// transfer stack contains (from top to bottom):
+	// Transfer stack contains (from top to bottom):
 	// - Token Filter
 	// - Packet Forwarding Middleware
 	// - Transfer
@@ -339,9 +339,9 @@ func New(
 		packetforwardkeeper.DefaultForwardTransferPacketTimeoutTimestamp, // forward timeout
 		packetforwardkeeper.DefaultRefundTransferPacketTimeoutTimestamp,  // refund timeout
 	)
-	// packetForwardMiddleware is used only for version 2
+	// PacketForwardMiddleware is used only for version 2.
 	transferStack = module.NewVersionedIBCModule(packetForwardMiddleware, transferStack, v2, v2)
-	// token filter wraps packet forward middleware and is thus the first module in the transfer stack
+	// Token filter wraps packet forward middleware and is thus the first module in the transfer stack.
 	tokenFilterMiddelware := tokenfilter.NewIBCMiddleware(transferStack)
 	transferStack = module.NewVersionedIBCModule(tokenFilterMiddelware, transferStack, v1, v2)
 

app/app_test.go

@@ -5,6 +5,7 @@ import (
 
 	"github.com/celestiaorg/celestia-app/v2/app"
 	"github.com/celestiaorg/celestia-app/v2/app/encoding"
+	"github.com/celestiaorg/celestia-app/v2/x/minfee"
 	"github.com/stretchr/testify/assert"
 	"github.com/tendermint/tendermint/libs/log"
 	tmdb "github.com/tendermint/tm-db"
@@ -39,6 +40,11 @@ func TestNew(t *testing.T) {
 	t.Run("should not have sealed the baseapp", func(t *testing.T) {
 		assert.False(t, got.IsSealed())
 	})
+	t.Run("should have set the minfee key table", func(t *testing.T) {
+		subspace := got.GetSubspace(minfee.ModuleName)
+		hasKeyTable := subspace.HasKeyTable()
+		assert.True(t, hasKeyTable)
+	})
 }
 
 // NoopWriter is a no-op implementation of a writer.