Skip to content

Commit ce68e8e

Browse files
authored
fix(iris): remove misleading 'All' log option and increase default max lines (#3683)
Drop the 'All' option from the LogViewer dropdown since it was capped at DEFAULT_MAX_TOTAL_LINES anyway. Increase DEFAULT_MAX_TOTAL_LINES from 10k to 100k so CLI users can fetch more logs when needed. Fixes #3672
1 parent a2d6a05 commit ce68e8e

File tree

2 files changed

+1
-2
lines changed

2 files changed

+1
-2
lines changed

lib/iris/dashboard/src/components/shared/LogViewer.vue

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,6 @@ const filteredLogs = computed(() => {
119119
<option :value="1000">1,000 lines</option>
120120
<option :value="5000">5,000 lines</option>
121121
<option :value="10000">10,000 lines</option>
122-
<option :value="0">All</option>
123122
</select>
124123
<select
125124
v-if="attempts && attempts.length > 0"

lib/iris/src/iris/cluster/controller/service.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@
8080
logger = logging.getLogger(__name__)
8181

8282
DEFAULT_TRANSACTION_LIMIT = 50
83-
DEFAULT_MAX_TOTAL_LINES = 10000
83+
DEFAULT_MAX_TOTAL_LINES = 100000
8484

8585
# Maximum bundle size in bytes (25 MB) - matches client-side limit
8686
MAX_BUNDLE_SIZE_BYTES = 25 * 1024 * 1024

0 commit comments

Comments
 (0)